-
Notifications
You must be signed in to change notification settings - Fork 16
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
Refinement #30
Refinement #30
Conversation
I just tried |
Where do those 28px come from? |
Right and left margin of 14px in the sidebar. One could even restrict that width to only inside the sidebar. |
Double check when this search bar is in the top bar. Physics change there. |
You can easily restrict |
Yes, this is what I thought too. |
Maybe 28px is not correct. margin-right and left: 9px and border-radius: 15px |
Looks like |
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.
LGTM now. Thanks.
Second try, make it dynamic in the sidebar only. This closes #30
@kwin Here is another try. This fits now into the sidebar, but will also narrow it for the topbar. I think those 7 percert are acceptable for the topbar.