sandstorm-io / sandstorm

Sandstorm is a self-hostable web productivity suite. It's implemented as a security-hardened web app package manager.
https://sandstorm.io
Other
6.76k stars 706 forks source link

900px break point for mobile is too high #2133

Open neynah opened 8 years ago

neynah commented 8 years ago

At 900px, all menu items fall into the hamburger menu which in it's current state much more difficult to use for navigation. We should only switch completely to mobile view at approximately 600px.

I say the break point is too high because:

  1. 600px-900px is plenty of space for topbar nav items as well as the sidebar(compact).
  2. Many users have dual windows displayed on their single monitor. There is a good chance their monitor does not fit 2 browsers >900px side by side. They have to rescale their window each time they want to navigate effectively.

Just above 900px: image image

Just below 900px: image image

zarvox commented 8 years ago

Perhaps we should auto-collapse the sidebar at sizes between 600 and 900 pixels (or when a window resize event puts you in that range)? That way the focus remains on the content, which might be totally at 700 pixels, but would have to switch to its mobile view at 500 pixels (assuming our 200-pixel navbar).

neynah commented 8 years ago

Perhaps we should auto-collapse the sidebar at sizes between 600 and 900 pixels (or when a window resize event puts you in that range)?

By collapse you mean to not display it all all or display the narrow form of the sidebar? I'd be +1 for displaying the narrower form between 600-900px. Which would then mean <600px would be the mobile view.

zarvox commented 8 years ago

By default, we would switch to the narrow form of the sidebar between 600 and 900px.

ocdtrekkie commented 4 years ago

Does this conversation need to involve detecting mobile browsers or DPI settings? In the age of crazy large and crazy high resolution phone screens, I am not sure pixels are a suitable measure of what interface should be displayed.

zenhack commented 4 years ago

@ocdtrekkie, the 'px' unit is not exactly a pixel:

https://developer.mozilla.org/en-US/docs/Glossary/CSS_pixel

See also: https://en.wikipedia.org/wiki/Device-independent_pixel