Olivero: wrap breadcrumb items.
breadcrumb items are now hidden after a specific width, these items should be visible and wrapped to the next line.
Adding screenshot for reference.
Issue fork drupal-3213153
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
- 3213153-olivero-wrap-breadcrumb
changes, plain diff MR !751
Comments
Comment #2
marcusvsouza commentedHere's follows a patch with some work to this problem but needs work!
Comment #3
sakthivel m commented@Gauravmahlawat breadcrumb items are goes certain width it's enable horizontal scrollbar.
it's working fine & attached screenshot
Comment #4
marcusvsouza commentedComment #7
indrajithkb commentedHi @marcusvsouza i have checked your merge request, here you just added the cahnges directly to this file "core/themes/olivero/css/components/breadcrumb.css" . that's why the custom commands getting failed. for this issue you have to do the changes on "core/themes/olivero/css/components/breadcrumb.pcss.css" and compile the css, and add both files.
And i have checked your changes manualy and i have seen one ponit, that is you have written
But we can't assume the length of the breadcrumb word, if you are adding a breadcrumb with more than 20 words, then we cannot say we can have the break after 7 breadcrumb-item. So please take care this point too.
Please go through this link this will help you to contribute on olivero theme.
Yes i know you are new to contribution.. Hope this will help you. Happy contributing... !!!
Comment #8
marcusvsouza commented@indrajith-kb thanks for the information, follows a new patch witch the necessaries changes. Please revise!
Comment #9
sakthivel m commented#9 Fixed Custom Commands Failed, recreated patch
Comment #10
marcusvsouza commentedHere's a patch correcting the erros from the previous one!
Comment #11
marcusvsouza commentedHere's a patch correcting the erros from the previous one!
Comment #12
mherchelThanks for the issue.
The design currently calls for a single line of breadcrumbs within an overflow container. At this point in the process, we're not making changes to the design unless absolutely necessary.