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

Reorder hamburger menu contents; eliminate Global Nav "More" dropdown #21

Merged
merged 3 commits into from
Feb 5, 2019

Conversation

benlk
Copy link
Collaborator

@benlk benlk commented Feb 4, 2019

Changes

Why

For #12

@benlk
Copy link
Collaborator Author

benlk commented Feb 5, 2019

The change in this branch in 85daad2 does put the contents of the global nav before the contents of the main nav in the hamburger-menu-controlled menu on mobile, but because that menu shares markup with the sticky nav, the sticky nav also displays the global nav contents before the main nav's contents.

Here's how it looks now:

screen shot 2019-02-04 at 7 01 10 pm

And here's how it looks with 85daad2

screen shot 2019-02-04 at 4 00 58 pm

screen shot 2019-02-04 at 4 01 18 pm

Questions for @khantasha:

  • Can the global nav come at the end of the hamburger menu instead?

@khantasha
Copy link

khantasha commented Feb 5, 2019 via email

@benlk
Copy link
Collaborator Author

benlk commented Feb 5, 2019

If we change the mobile nav, the sticky nav also changes. Here's what it looks like with the global nav contents positioned after the main nav contents:

screen shot 2019-02-05 at 11 03 45 am

Note that the former "more" menu contents are now just part of that bar's contents; the "More" menu shown is the overflow menu item that appears when the menu would be too long to fit within the viewport.

screen shot 2019-02-05 at 11 48 26 am

The sticky nav is the mobile nav; the display presentation changes based on the screen width.

Is this okay? @khantasha

@benlk benlk self-assigned this Feb 5, 2019
@khantasha
Copy link

khantasha commented Feb 5, 2019 via email

@benlk benlk merged commit 8886f8a into staging Feb 5, 2019
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

Successfully merging this pull request may close these issues.

2 participants