Skip to content
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

Navigation design issues #3184

Open
1 of 6 tasks
jancborchardt opened this issue Sep 6, 2022 · 4 comments
Open
1 of 6 tasks

Navigation design issues #3184

jancborchardt opened this issue Sep 6, 2022 · 4 comments
Labels
1. to develop Accepted and waiting to be taken care of bug Something isn't working

Comments

@jancborchardt
Copy link
Contributor

jancborchardt commented Sep 6, 2022

Just tested with tech-preview again, Mail version 2.0.0-beta.2 (If any of them are related to the Vue components instead, please move them over there instead.) @GretaD @ChristophWurst

  • There is too much spacing in a lot of places Fix double <ul> in nav nextcloud/mail#7194
    • Too much to the left/right and top for the regular navigation entries. It looks fine for the "Mail settings" entry
    • There is too much space above the "Mail settings" entry when it’s not open
    • The navigation has space on the right towards the right edge of the navigation, shouldn’t have
  • The "Mail settings" button is not selected/active, thus should not be highlighted / not have a background
  • The mail settings content area is cut off at the bottom before reaching the actual bottom of the nav

Mail nav

image

@jancborchardt jancborchardt added bug Something isn't working 1. to develop Accepted and waiting to be taken care of labels Sep 6, 2022
@ChristophWurst
Copy link
Contributor

Most of the styling comes from @nextcloud/vue.

@jancborchardt jancborchardt transferred this issue from nextcloud/mail Sep 6, 2022
@jancborchardt
Copy link
Contributor Author

Moved over to Vue then. :) cc @marcoambrosini @GretaD @raimund-schluessler are you aware of this?

@ChristophWurst
Copy link
Contributor

Well well well how the turntables. That is partly Mail's fault.

@ya-d
Copy link

ya-d commented Sep 10, 2022

I don't know if this is related to Vue components or Photos. But there's too little padding over there:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants