-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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 app menu makes external links very hard to use #3959
Comments
We could show some icons like the avatar fallback with the first character of the external site name instead of using the external icon. Then the users would at least be able to differentiate between different sites. Input from @nextcloud/designers would be good as well. |
Sounds nice, but I would do then the avatars in monochrome to not distract too much. So basically a white circle with a transparent letter in .7 opacity or so. |
In the past I thought it would be great to add favicons here. That of course looks bad because they are colored on a colored header. Gladly Apple pushed the So that’s something we could do. However I heard that mostly organizations embed sites within their system there, hence having the same favicon. For that then a monochrome version of the avatar fallback might be best. Another option: Be able to choose from a set of icons – like all the ones we have in |
You can already upload icons yourself. But yeah I thought there might be an easier way. Any strong opinion on the "move every navigation which is lower then 80 into the dropdown? If yes, we can also simply close this PR and then they just need to generate icons to use. |
@juliushaertl what do you think about
|
Too late for 12. |
@juliushaertl @nickvergessen wasn't this done already? I faintly remember something. |
Well we at least show the page name now directly on hover with the tooltip, and not only after 5sec waiting. So "fine by me" |
I just wonder if we can do anything against this.
Yes people should use different icons, but maybe we can also do something else for this special app?
@nextcloud/theming @juliushaertl
The text was updated successfully, but these errors were encountered: