-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Rethink Learn section of Navbar #3984
Comments
Maybe a more button that could show the rest? |
@minimalsm hey can this be a hacktoberfest issue? |
hello |
adding a scroll ? |
Hey everyone. This issue requires design before being worked on (i.e. if you have a proposal feel free to comment here). Some sort of re-organization of the navbars is probably required for this. |
Marking this as blocked - I think we should re-think this as part of an overhaul of the learn page more broadly: My gut take as of now is we can build out a landing page for the Learn section of the site that acts as a hub & provides a hierarchy of topics & subpages, similar to how we have the "Developers' Home" within the "Developers" dropdown menu. |
Noting this is also a bug on smaller screens/browsers - the dropdown nav is cut off with no ability to scroll: Screen.Recording.2021-11-16.at.06.23.08.AM.mp4@minimalsm perhaps a short term fix would be to remove some of these pages from the dropdown? e.g. I think at least these could be removed - they're less important pages & are linked to from the footer & elsewhere:
|
I have a proposal for the design. Here are my thoughts:
Learn sectionAnother nav dropdown |
@minimalsm thoughts? |
Thanks for the suggestion @aguscruiz! I like your proposal 😄 |
This issue is stale because it has been open 45 days with no activity. |
Multiple folks have also reported this in Discord: |
Basic/AdvancedQuick win would be properly grouping and nesting different ideas. Since this tab is about learning, it makes sense to group ideas into Basic and Advanced (people shouldn't read "Blockchain Bridges" before "What's ETH?"). Here's one grouping: Learn
Then you can lay out the nav horizontally, which each section next to eachother to save on vertical space: Learn You can add MobileThe site has a pretty hefty nav overall. For mobile, it makes sense to ditch the typical pop-out list and for a full width/height, global nav modal. It would have 2 columns to save vertical space. Scrolling BugSlapping a |
Assigning to @konopkja 😄 |
This comment was marked as spam.
This comment was marked as spam.
This issue is stale because it has been open 45 days with no activity. |
@minimalsm @konopkja I imagine this can be closed now. |
Solved in #6778 - thanks @konopkja & @pettinarip! |
Is your feature request related to a problem? Please describe.
The number of links in the 'Learn' section of the main navigation is outgrowing the initial design.
This looks fine for now but very soon we'll be adding 3-4 new pages. Aesthetically this doesn't look great but also runs into Hick's Law problems - reducing the likelihood users will find the information they are looking for and reducing the user experience.
Describe the solution you'd like
Open to suggestions. Just wanted to highlight this issue.
Want to contribute?
We love contributions from the Ethereum community! Please comment on an issue if you're interested in helping out with a PR.
The text was updated successfully, but these errors were encountered: