-
-
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
"Create new" Icon is not in line with sidebar #40550
Comments
Hey @pschopen , Can you please explain a bit more where it is causing an error so that we can discover and fix it? |
It's not an error in the first place. It's just not ideally designed so something for the design team to be decided. I guess the best option would be to apply the same margin to sidebar and main container. Then everything would be aligned properly. Maybe @nimishavijay can have a look? |
Yeah I'll set it's margin and padding can you give me more details where it button appears. |
Yes, makes sense! The navigation toggle, breadcrumbs component and the new item button should all the middle aligned :) |
Thanks I'll try to fix it. |
Where i can find local setup readme or somthing. |
Hello, Is this issue still open to be resolved or can it be assigned to me? |
@ASP124 it is still open, you can work on it if you would like! Resources on contributing code can be found at nextcloud.com/developer :) |
Hello! I'd like to get involved and help as a designer but I'm very new to this. Could you please tell me how exactly can I do it? Where do I find initial designs to correct/change them in Figma? |
@5gorsky you can find more about contributing to design at https://nextcloud.com/design/ :) |
Fixed for 28 |
As my previous issue got closed and this is actually a different problem and rather something for the design team to do, I decided to create a new issue for this.
Before having the redesigned "new" button I didn't realise that margin doesn't match the sidebar, now it's obvious:
Originally posted by @pschopen in #40510 (comment)
The text was updated successfully, but these errors were encountered: