-
Notifications
You must be signed in to change notification settings - Fork 120
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
Responsive views: Key breakpoints and design cornerstones #1820
Comments
My only remark about this is that on desktops it's usually the vertical space that is more limited than the horizontal space. Eg: on laptops you'll have plenty of horizontal space for the sidebar+content, but not to show the full contents of a given page and then you'll require vertical scrolling. |
This was referenced Dec 22, 2018
Status on these. Launcher
Inside Wallet
|
This was referenced Jan 21, 2019
This was referenced Feb 25, 2019
Closed
This was referenced Feb 25, 2019
This was referenced Nov 6, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Preview: https://xd.adobe.com/view/4f6828c6-71b4-491f-5224-579b840b2743-3959/
Specs: https://xd.adobe.com/spec/3bd71ab7-4a8e-43d4-5ab8-2ad3dd3500ee-4de6/
Bit of insight and instructions:
When implementing these, be aware the design approach for decrediton has been desktop-first so far, thus the responsive views for smaller sized viewports will first focus on getting a kind of basic, usable and visually hygenic solution on the table. Focusing mainly on a minimal responsive approach, and keeping any changes/adapations as simple and quick as possible. Obviously some modules/elements can't simply fit or scale, so in those instances some adaptive solutions will be needed.
The work can be started by defining the key viewports. The largest (and current) is 1180 px, which has a content area of 740 px. Notice that currently when re-sizing Decreditons window to even larger sizes, in some of the views elements scale with it. This should not happen. Instead if the view is scaled further in width all elements should stay within the content area. When the navigation is closed, the content area should retain the same spacing what it has by default between the navigation, and change it's position slightly to the left ( following the navigation (e.g. slide 2)).
Second smaller breakpoint is 768 px, with a content area of 668 px. This is a common tablet size. When the user scales down to this, nav. menu should trigger closing into smaller state (or be closed by default if opened in smaller window). Also when the user expands the nav. menu and clicks on another view, the nav. should then close (when transitioning to the next view). When the nav. is expanded, nav. menu should stay on top of the content area (kind of as right now) without affecting any elements there.
Smallest mobile sized breakpoint is 375 px with a content area of 355 px. In this instance, the navigation has a new variant. The mobile nav. is locked in the bottom corner of the viewport, and by default in a closed state. Due to size limitation, the closed menu contains only 4 key items (as well hamburger for opening complete navigation) those being: Overview, Transactions, Governance and Settings. When opened, the complete navigation covers entire viewport, as well displays block sync status. Menu items in the full nav. variant are of same size as in other viewport sizes, although have different vertical spacing (and selector height) for better fit. Menu icons in closed state are slightly larger for ergonomic reasons (w/ 24px bounding box). Content area always continues vertically below the navigation.
As a follow-up to this issue, each view will receive a visual design example, presenting and describing how to elements should be layed out and confirming whether there's any adaptions needed. These will be posted as separate weekly issues.
Also you may notice the action buttons are right aligned. These are currently in progress
Decrediton: Ok/cancel and buttons right-alignment dcrdesign#64 and will be posted as a separate issue once covered in all views (so should not be addressed yet).
The text was updated successfully, but these errors were encountered: