-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
New share icon in breadcrumb nav #3564
Comments
cc @nextcloud/designers |
Simply out of space reasons. But the spec was always that the breadcrumbs also show if a parent folder is indeed shared, as an indicator. |
fix #3564 Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>
Pull request to change the default icon to icon-more (3-dots) at #6143 :) |
fix #3564 Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>
Actually I don't really mind the icon, I think the opacity is different enough to understand it.
But for this, I think this is a big issue. |
I am really hesitant to add a different icon here.
Unless the issue is more pressing or we come up with a really good solution, I would say we close this? Also, the change to possibly show the sidebar by defaut on big screens will make this less of an issue: #7138 |
The new share icon in the breadcrumb navigation was probably meant to speed up everyones workflow. As a user I have to say that I find it very confusing.
not shared folder
shared folder
My points of criticism being:
My suggestion:
Well, I don't have one at the moment. Personally, I see this as a big cause of confusion within the interface, because of the three inconsistencies mentioned. I'm interested, if anyone agrees to that – or if maybe someone might be able to come up with a solution (that is not removing the icon again).
The text was updated successfully, but these errors were encountered: