-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Switch to ant sider/content Layout modules, to help style sidebar #188
Conversation
5cdf9b7
to
c3e9e2a
Compare
Fixes the problem of the sidebar not extending all the way on long pages. Signed-off-by: Risha Mars <mars@buoyant.io>
…item was selected Signed-off-by: Risha Mars <mars@buoyant.io>
c3e9e2a
to
3b56dbf
Compare
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.
⭐️ 😻 Huge improvement!
web/app/css/sidebar.css
Outdated
background-color: #091B39; | ||
color: white; | ||
height: 100vh; | ||
min-height: 200vh; |
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.
I think we should change this to min-height: 100vh;
. Otherwise we'll display a vertical scrollbar on short pages that would not otherwise have a scrollbar.
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.
ooh, thanks! will do
Signed-off-by: Risha Mars <mars@buoyant.io>
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.
This looks great, interesting before/after 😻
* Switch to ant sider/content Layout modules, to help style sidebar This fixes the problem of the sidebar not extending all the way on long pages. * Fix a bug where the autocomplete options weren't being reset when an item was selected
This picks up the following commits: * 0fe8063 replace with (#2370) (#201) * 1ea7559 Minor cleanup in the config tests (#188) * d0ef56b Update *ring* to 0.14.6 (#197) * c54377f fs-watch: Use a properly sized buffer for inotify events (#195) * 23e02a6 Update Router to wait for inner poll_ready before calling inner call * 2de8e9b Update metrics quickcheck to 0.8, and hyper to 0.12.24 * d1bbd4b make: Optionally include debug symbols with builds (#193) * 738a541 Fix compilation warnings in fs-watch (#192) * 6cc7558 Apply rustfmt (#191) Signed-off-by: Ivan Sim <ivan@buoyant.io>
Signed-off-by: Alex Leong <alex@buoyant.io>
Fixes #72
Before:
After:
Signed-off-by: Risha Mars mars@buoyant.io