-
Notifications
You must be signed in to change notification settings - Fork 22
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
Left hand menu headers ux enhancement #16
Comments
@PhilBarrett @tony-shannon some suggestions - we can, for example, reduce spacing between the items (this way we would be able to display more elements) What's your thoughts, Tony? |
@simongamester I believe has been thinking about this too. He might have a view too. |
Yes, we've discussed this before. Would suggest adding a native scrollbar to the side nav, only visible when the height of the side bar is greater than the window. Would this not just be an overflow-y: scroll? This slide in menu seems to do this: If the browser window is too narrow, scroll bars are automatically added: I like that they've added a 30px divider to the right of the menu, which is replaced by scrollbars when needed. Keeps the layout consistent: |
Hi Dmitry
Happy to try Simons suggestions
thanks
T
…On 13 June 2017 at 16:18, simongamester ***@***.***> wrote:
Yes, we've discussed this before. Would suggest adding a native scrollbar
to the side nav, only visible when the height of the side bar is greater
than the window. Would this not just be an overflow-y: scroll?
This slide in menu seems to do this:
https://callmenick.com/_development/slide-push-menus/
If the browser window is too narrow, scroll bars are automatically added:
[image: screen shot 2017-06-13 at 16 14 13]
<https://user-images.githubusercontent.com/8059219/27089802-6111e7b4-5053-11e7-941e-bdc6ac06b1af.png>
I like that they've added a 30px divider to the right of the menu, which
is replaced by scrollbars when needed. Keeps the layout consistent:
[image: screen shot 2017-06-13 at 16 15 15]
<https://user-images.githubusercontent.com/8059219/27089861-87747fca-5053-11e7-9af9-41a20a212805.png>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#16 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AA5X3MkW4hsLK2klH4mcp_pmkMxrreqJks5sDqhggaJpZM4NzwmA>
.
|
Depending on your screen size/resolution the full left hand menu doesn't fit on a single screen. There is nothing to show that there are more menu items to view in this scenario, preventing the user having a full picture of the patient (see screenshots)
The text was updated successfully, but these errors were encountered: