New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Style breadcrumbs #1595
Comments
I'm proposing a more simple solution than you've outlined above @noahmanger — based on if it can be clicked or not instead of revealing so much about our structure. This might be unpopular 😁 but here goes: The system is:
These combos do pass requirements for 508 color contrast ratios :) A thing we probably want to think about and I've included above, is truncating very long page names One change I'd consider making if we feel it's necessary is making the current page a bit more visually distinct in a subtle way. Mocked up below to demonstrate, but I do prefer the simplicity above^ |
I meant to respond earlier: This looks great. I love the simpler approach. I think it might be good to make the current page bold, though. Or at maybe white? What do you think? And +1 to truncating long names. That's easy and a good idea. Just gimme the word and I can move this to implementation. |
Bold / Grey Italic / White White / Bold If you agree, @noahmanger , feel free to take it from here! |
I've been following this quietly, but I did want to make sure that I explicitly said I am 👍 with truncating the text |
Per this morning, going with simply showing the current page as white, but not bold. Moved to implementation. |
Usability testing revealed the breadcrumb pattern clearly works. Users noticed them, they understood where it put them in the information hierarchy of the site, and they knew how to use them to navigate up the hierarchy.
Here is example of the logic of the breadcrumbs:
Note: We will not show siblings, and we need a way to visually distinguish non-link parent category titles from parent pages that are links.
Once the designs are complete, we will implement in fecgov/openFEC-web-app#1154
The text was updated successfully, but these errors were encountered: