-
Notifications
You must be signed in to change notification settings - Fork 22
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
[feat] <rh-breadcrumbs>
element
#1458
Comments
Figma Design Spec for |
Wondering if this one will have guidance for mobile. For example, truncating works when there aren't too many levels like resource pages on WWW, but when there are more levels it would be useful to keep them and maybe have them wrap? (see screenshot) Or even if we should keep them on mobile or not. Customer portal drops theirs. |
@coreyvickery: do you have guidance for breadcrumbs on mobile? Generally, I think having the text wrap is standard for most orgs. |
@adamjohnson I would expect the Large viewports:
Small viewports:
Even smaller viewports:
I would avoid having the Let me know if you need further guidance like spacing! |
@adamjohnson Or we could try another approach similar to IBM Carbon where they are breaking each step at a given container size. What do you think would be best? |
Wrapping is fine.
Can do. Note: the design spec truncates the final breadcrumb item if it's over 27 characters.
Would you want "Home"/"About" to truncate if they are longer as well? |
@adamjohnson No, I think the previous steps should be visible. I am fine with the last step being truncated because the headline is literally right below. Instead of the above, I would think it would look like this:
Maybe two lines max and then truncation no matter what. |
for longer paths, i wouldn't want to only limit them to two lines also, if it hasn't been added yet... would be nice to have guidance around how far back to have the path go, for example all the way to "home," and placement / alignment of breadcrumbs on the page -- they're all over the place now |
WIP Deploy Preview links:I wanted to put these WIP Breadcrumb deploy preview links here so that people can see what this looks like now:
The only truncation happening at the moment is on the current page when it's below 992px. The breadcrumbs Other libraries:Here's breadcrumb component examples from other WC libraries that don't truncate and just wrap text: Looking at Mark's examples (above), that pattern is interesting from a design and engineering point of view; however, I'm not sure how useful that is to the user since, at smaller viewports, you only get 1-2 letters of a word/words. Atlassian has an interesting take on breadcrumbs, which is somewhat similar to Patternfly. I personally like Atlassian's implementation better, but YMMV. Proposed solutionsWith this in mind, here's the solutions that have been proposed:
My vote would be to go with option one or two, but am open to other ideas. |
@adamjohnson @wesleymiles Thanks for your input, we can show more lines on mobile. Funny about that Atlassian link, I was typing this last night. What is the longest path you have seen? Would we ever expect to see something like six links or something? Should we set a link number limit before truncation? Something like this:
Or even truncate the Home link:
|
also like atlassian's solution. could we set auto-collapse based based on the # of lines? or maybe that's what you mean with your proposal #3 @adamjohnson. |
@adamjohnson @wesleymiles I'm going to start a separate Figma file to explore creating more design options. I'm really hoping this can be included in the |
I'm prototyping the Atlassian solution now. Would love to see what you come up with, Corey. |
@adamjohnson Very small updates. Light theme
|
@adamjohnson Hover colors in case you need them. Light theme
Dark theme
|
Looking at the Figma mockup, I'm thinking we should get out an MVP following the "Responsive approach 1". If we can get "Responsive approach 2" working in the future with a container controller / query, that would be ideal, and we could upgrade to that in a future version. |
@adamjohnson How's it going with this? |
Check out the DP at #1535 👍 |
@coreyvickery (and @adamjohnson if you have time), here's a mock up of the breadcrumb docs. Let me know if you see anything that should be included, removed, or edited! |
We need a Breadcrumb component for use across applications and websites.
The good news is that it seems we have a very closely aligned pattern across PatternFly and our web properties already. We will need to have a design spec created for documentation and made available in Figma.
Steps
Resources
<pf-breadcrumb>
element patternfly/patternfly-elements#2688Breadcrumb best practice
From WAI's breadcrumb pattern:
The text was updated successfully, but these errors were encountered: