-
Notifications
You must be signed in to change notification settings - Fork 13
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
Header (global and country) shouldn't change to tablet view for screens larger than 1024px #916
Comments
This will require a flexible solution for displaying menu items (particularly more than 4). |
Here is the link to the final Figma mock up. Thanks |
Taking over for development |
Because the css work to fix 915 was so close to some of the fixes I had to do for this task, I just addressed it all in this issue. |
…ldnt-change-to-tablet-view-for-screens-larger-than-1024px #916 header global and country shouldnt change to tablet view for screens larger than 1024px
@mrelevo , please verify |
verified in Dev |
@paboden we just noticed that once you get into view in a browser <1440, the UNDP logo is no longer clickable. It becomes clickable again <1024. This only occurs in the Global header |
@paboden , there is another issue appeared, I'm afraid - now we are missing logo navigation (shrinking) when scrolling down the page: try https://916-fix-global-logo-link.design-system.pages.dev/?path=/story/templates-global-homepage--global-homepage for the viewport between 1024 and 1440 wide |
@paboden , currently entire header is being shifted up on scroll there, this is not correct, only logo should contract/expand on scroll. Check the implementation for big screen (>1440) to use as an example and here is the screen recording from viewport in question with that undesired shift: Templates._.Global.homepage.-.Global.homepage.Storybook.-.Google.Chrome.2023-03-06.11-38-54.mp4 |
@paboden , after last change the issue I reported is gone, but now Language switcher is shifted slightly up, the text is not aligned with the menu items anymore: |
@paboden here's another approach at showing the misalignment issue. Note how the language switcher is sitting higher than the menu items. You can also see this change in Chromatic |
@paboden another small regression: For the Article template, the progress nav bar disappears between 1024px to 1440px. Screen capture of the issue: progress.bar.mov |
…and xlarge screen states. Addressed issue with Language selector's text not lining up with other text (menu items) in the header. Addressed this issue in both global header and country header
@theadamparker This is ready to review again here - #1148
|
#916 -- Fix to logo z-index on large screen to make clickable again
@mrelevo , over to you for verificaiton |
Hi @paboden, all the issues have been resolved. However, a new one has been introduced. The language selector option now shows below the header See correct behavior here And its fine as a standalone https://develop.design-system.pages.dev/?path=/story/components-ui-components-language-switcher--language-switcher |
#916 -- fixed vertical offset language dropdown
@mrelevo , over to you |
looks good dev |
verified in storybook and Drupal DEV |
Currently the headers lose the mega menu hover effect and switch to the collapsed menu with hamburger icon when the browser is <1440px. This should happen only for browser width <1024px. For everything >=1024px the full menu with mega menu hover effect should be used.
UPDATE: flexible top level menu is separated to its own issue: #1098 based on the agreement with Chapter 3
The text was updated successfully, but these errors were encountered: