-
-
Notifications
You must be signed in to change notification settings - Fork 956
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
SAK-49165 Portal site header should include links to parent sites #13286
Conversation
ottenhoff
commented
Feb 6, 2025
•
edited
Loading
edited
![Screenshot 2025-02-06 at 3 01 07 PM](https://private-user-images.githubusercontent.com/810505/410619197-19ef5001-ed62-4ce2-bf24-f496ef361425.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzk2MzkyMDYsIm5iZiI6MTczOTYzODkwNiwicGF0aCI6Ii84MTA1MDUvNDEwNjE5MTk3LTE5ZWY1MDAxLWVkNjItNGNlMi1iZjI0LWY0OTZlZjM2MTQyNS5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwMjE1JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDIxNVQxNzAxNDZaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1mNzEwYWEzZmRjYTMyZmFhZjVhYTZjZDM3Y2Q1OGJmOTdiODBlYmU3MWIxN2M3NzdlNjUxYzEzZmEwZWZmNmY2JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.QIi7ZuSMjp7RimhwjQsHJhgzNfJg4qYGdgoH2ySVlhI)
Are all the levels on the breadcrumb is required, its looks bit crowded to me? and what about smaller devices? |
This is the most extreme set of parent sites i've ever seen. i think just one is the most common use case.
it doesn't appear at all on smaller devices |
@ottenhoff It's a good question though. How does it wrap if long site names are used? Do you clip the site name, or just let it wrap? |
It wraps. Same behavior as current. Note that I'd love to see this feature disappear. I think truly hiding it by default makes the most sense. Legacy users can continue to use it by setting a property. This is a classic 1% user feature |
@ottenhoff Totally agree. 1% user feature. |
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.
Cheers