Skip to content

Conversation

@jproberson
Copy link
Contributor

PR Checklist

  • - Unit and/or e2e tests for the changes that have been added (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Refactoring (no functional changes)
  • Documentation
  • Other:

What is the new behavior?

trying to handle breadcrumbs for mobile better

Does this PR introduce a DB Schema Change or Migration?

  • Yes
  • No

Git Issues

Closes #

What happens next?

Thank you for the contribution! We will review it ASAP.

If you need more immediate feedback you can reach out to us on Discord in the Community Platform development channel.

@github-project-automation github-project-automation bot moved this to 🆕 Backlog in Core Team - BAU list Dec 30, 2025
@jproberson jproberson added the Review allow-preview ✅ Has received manual check for malicious code and can be safely built for preview label Dec 30, 2025
@cypress
Copy link

cypress bot commented Dec 30, 2025

onearmy-community-platform    Run #8348

Run Properties:  status check passed Passed #8348  •  git commit b5d7f0761b: remove scrollbar for breadcrumb
Project onearmy-community-platform
Branch Review fix/4508-breadcrumb-overflow
Run status status check passed Passed #8348
Run duration 07m 03s
Commit git commit b5d7f0761b: remove scrollbar for breadcrumb
Committer Jacob Roberson
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 2
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 91
View all changes introduced in this branch ↗︎

@dalibormrska
Copy link
Member

Looks awesome! That's exactly the behaviour that we want!

One last thing would be to hide the scrollbar, but keep the scrolling functionality. This worked for me in devtools on chromium:
image

Here I quickly checked what other browsers might need, but idk:

css-8k4646 {
  scrollbar-width: none; /* Firefox */
  -ms-overflow-style: none;  /* IE and Edge */
}

css-8k4646::-webkit-scrollbar {
  display: none; /* Chrome, Safari, Opera */
}

@mariojsnunes
Copy link
Contributor

Here I quickly checked what other browsers might need

Note that no one uses IE or the old Edge anymore, so that line isn't needed

@jproberson jproberson marked this pull request as ready for review December 30, 2025 15:21
@jproberson jproberson requested a review from a team as a code owner December 30, 2025 15:21
@jproberson jproberson added Review allow-preview ✅ Has received manual check for malicious code and can be safely built for preview and removed Review allow-preview ✅ Has received manual check for malicious code and can be safely built for preview labels Dec 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Review allow-preview ✅ Has received manual check for malicious code and can be safely built for preview

Projects

Status: No status
Status: 🆕 Backlog

Development

Successfully merging this pull request may close these issues.

4 participants