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

Reduce vertical spacing on submenus #45

Closed
mikemccamon opened this issue Aug 21, 2019 · 4 comments
Closed

Reduce vertical spacing on submenus #45

mikemccamon opened this issue Aug 21, 2019 · 4 comments

Comments

@mikemccamon
Copy link

Check out how "loaded" menus look on https://www2.owasp.org/ Need more compact vertical line spacing.

@scorchingoyster
Copy link
Owner

It's a little heavy. I think I will tweak both the line-spacing and the font-size to get them look more breathable—it seems to occupy a lot of vertical space when expanded as well, so that should help.

@mikemccamon
Copy link
Author

Sounds great!

@scorchingoyster
Copy link
Owner

How is this? About 20% reduction in height; I changed the font-weight but lowered the size, and then decreased the vertical spacing.

Also showing the submenu separator at the bottom for #46.

i_view64_POqkEHhfUL

@mikemccamon
Copy link
Author

For some reason the font is not less readable in the picture on the right. I'm almost thinking that just reducing the line spacing on left would get us there. (Also looks like font on right is bold now(?) I liked your original selections just not so much negative space in line spacing IMHO

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants