-
Notifications
You must be signed in to change notification settings - Fork 6.2k
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
fix: added ellipsis for long content in breadcrumb #6794
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Signed-off-by: Claudio W <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alrighty, awesome contribution! Thanks for fixing this :)
Lighthouse Results
|
hey @ovflowd |
Hi @TenzDelek, thanks for investigating! Unfortunately I don't have the time/capacity to look into this now, maybe someone from the Website team can? (cc @nodejs/nodejs-website) |
@TenzDelek now it's not centered anymore |
gap-5; | ||
w-full | ||
gap-5 | ||
pl-2; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pl-2; | |
px-2; |
It's allow to have same padding left and right
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will implement that right off. also about the breadcrumb being on left, I believe that is due to making it full width. should we keep it or make it as default? the purpose of this was there was noticeable shifting of the breadcrumb when different text length occurs. so keeping it at a same starting point we can reduce that shifting.. :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can see arguments either way, but since people are not really seeing the centered breadcrumb move (since it's at the bottom) it is a small concern to me. I'd rather it be centered, just as an opinion (and to preserve the original design's integrity), or if it is anchored to the left, it has padding similar to the content
![image](https://private-user-images.githubusercontent.com/298435/336854331-7fce66bd-7190-4b8f-80f8-e108f01cca5a.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA1NzE3NDksIm5iZiI6MTcyMDU3MTQ0OSwicGF0aCI6Ii8yOTg0MzUvMzM2ODU0MzMxLTdmY2U2NmJkLTcxOTAtNGI4Zi04MGY4LWUxMDhmMDFjY2E1YS5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzEwJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcxMFQwMDMwNDlaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1hY2E3MWFlMzBkZWFkNDJmMTdkZGFhODY4MTY1NmY1MmE1Mjk0NTRjNTcxODg1NThkMThiYWJmYTUzMzdjZTg0JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.h99tC1C9n1aIPkwGPwZ2H-KBII82zpWenl_U4ywtfOw)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bmuenzenmeyer appreciate your input on this. I too see this issue in either way, but if it affecting the integrity of the original design, then definitely i will shift it back to the center as mention by both @AugustinMauroy and you:) .
p.s I am still figuring out about our main conversation which is to make ellipsis on the longer text format. if any of the members have any navigation on that, it would be helpful :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bmuenzenmeyer can you follow up here? Are you fine with this getting merged as it is? :)
This PR will need to be rebased or recreated now that #6850 merged. |
Description
added ellipsis for the long content in the breadcrumb to improve the layout shifting and in some case, breaking of the breadcrumb into next line.
Validation
before:
![image](https://private-user-images.githubusercontent.com/122612557/335948635-871323bc-0985-48af-92bb-d7f73efef52e.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA1NzE3NDksIm5iZiI6MTcyMDU3MTQ0OSwicGF0aCI6Ii8xMjI2MTI1NTcvMzM1OTQ4NjM1LTg3MTMyM2JjLTA5ODUtNDhhZi05MmJiLWQ3ZjczZWZlZjUyZS5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzEwJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcxMFQwMDMwNDlaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT0xMDgxNjgyOTdmOTI5NThjMmI0MGI2NDZiZTIzOTRmOThhZTNjOGM0ZDIzZWY0YTg5MTVhNGU5M2Y2MmI5MGViJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.5LTK5gmStvc45G6ULbeM6GJeR-ANDrJB9tsSg3pDooA)
after:
![image](https://private-user-images.githubusercontent.com/122612557/335948916-75df5705-b141-4e43-bb96-8ef388528762.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA1NzE3NDksIm5iZiI6MTcyMDU3MTQ0OSwicGF0aCI6Ii8xMjI2MTI1NTcvMzM1OTQ4OTE2LTc1ZGY1NzA1LWIxNDEtNGU0My1iYjk2LThlZjM4ODUyODc2Mi5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzEwJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcxMFQwMDMwNDlaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1mZjg3MDRlNDQxNzJmMWJlMTc4MDgxNGI3YTA0ODljMWQ3MTgwOGRhMGQ2M2ZkZGY5Mzc2ZjU2MmNjYWNjZTE0JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.hNHMEJawbwRSLnx9Az6uEkiL-vTolwcCuOtBqTWXYck)
Other Changes
Related Issues
Fixes #6754
Check List
npm run format
to ensure the code follows the style guide.npm run test
to check if all tests are passing.npx turbo build
to check if the website builds without errors.