-
Notifications
You must be signed in to change notification settings - Fork 63
Menu and breakpoint improvement in new header #2009
Comments
Subscribe to Label Actioncc @WordPress/gutenberg-design, @WordPress/openverse
This issue or pull request has been labeled: "🖼️ aspect: design"
Thus the following users have been cc'd because of the following labels:
To subscribe or unsubscribe from this label, edit the |
The proposed footers and drop-down menu for mobile look great! |
Thank you for the screenshots, @panchovm! This is very helpful to visualize what the full experience will be like. |
@panchovm These look great so far. Can you add mockups of the full version that includes a hover and focus states for the links including the following edge cases?
|
Lots to like here, nice work as always. One small niggle I have personally is with the "external link" icons. When intermixed with the other items, they clash between being both first and second class citizens at the same time. I'd love to find a way to remove them (which shouldn't block any efforts here, but is mainly shared for posterity). I understand they are there because technically they link to openverse.engineering. But I'm curious if we could just omit the icons still: if I've decided to click "API", I'd probably expect going somewhere technical, and I can always press "back". Screenreaders would still read the link to be external, of course. Alternately, you could create a landing page on the main site to link to, without the external link — a "meta get started" page, and then link externally from there. This might also be a better experience for menu items intermixed with consumer-first items. |
I like the idea of landing pages to introduce certain topics, but I also like how Get involved takes you to the WordPress site and reinforces the project relationship. It is definitely food for thought, which visitors' data can enlighten once we start collecting analytics. |
I did not understand your request correctly @sarayourfriend, but I added a new section called Link states in the Header file where I put the states you mention. Here is a screenshot for a quick glance. I did not add a Link states section in footer as it lives in the Design Library. Nonetheless, here is a mockup echoing the header example. NoteYou can see the Link component's details used in Header and Footer components in Figma. Select the Footer component, open the internal layers, select the Link component, and open the component in the Inspect tab. 1. Select the "Link" component in "Footer" component2. Open the "Link" component from the Inspect tab |
That is perfect, thank you, Francisco. |
@panchovm, I wanted to confirm with you before closing this issue. We can close it now, right? |
Yes. We can close it. The outcome looks excellent ❤️ |
Problem
After the implementation of the new header, there are three needs we need to tackle as improvements.
1. Internal pages
The current site has internal pages that were not included in the new header design as the content discussions did not settle and ended up in a insufficient list of item in the new mockup.
Due to the above, @obulat revisited the content of each page and shared with me questions and thoughts.
2. Active state
The new header did not consider an active state for the page link.
3. List extension in breakpoints
On the other hand, the page list overlaps Openverse logo in
md
breakpoint whereas in cases with available space, the full-page menu covers the device screen unnecessarily.The second point does not show a UX problem, but it does have room for reaching a cleanest design.
Issues related
Proposal
Menu pages
After reviewing the content of each page in the menu, I propose having these pages in the following order:
The order proposed goes from generic to specific: The project story (About); describe the content displayed (Licenses, Sources); tips on searching (Search guide); how to contribute at a more expert level (Get involved, API); and finally the privacy approach and asking for feedback.
I removed the “External sources” page from the list as I think we can put the content in the About page. It relates to the Openverse ethos of why we want to highlight sources not connected yet; it can also be moved to the Sources page. But linking a page from the header to this feature looks vague and needs more context.
Since the menu has a mirrored version in the footer component (content version), the proposal includes a footer update as well.
Header internal
Footer content
“Link” component
For the active state needed, I created a new "Link" component with resting, hover, and disabled states for its active and inactive versions. The component can be reused in other areas as is consistent with other actions across the site.
Breakpoint change
To show all pages correctly in every breakpoint, the design proposes using the burger version in both
sm
andmd
breakpoints. However, the full-screen menu is shown differently inmd
to tap into the viewport's available space.The menu popover has Popover item components to be consistent with the Design Library’s elements available. At the same time, the label of each item is based on the “Link” component mentioned above to keep consistency.
< sm
breakpointsm
,md
breakpointsMockups
Implementation
sm
andmd
#2013The text was updated successfully, but these errors were encountered: