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

Header (global and country) shouldn't change to tablet view for screens larger than 1024px #916

Closed
theadamparker opened this issue Mar 30, 2022 · 20 comments · Fixed by #1113
Assignees
Labels
design This issue requires design work before development can begin enhancement New feature or request extdev outsourcing to external vendor

Comments

@theadamparker
Copy link
Contributor

theadamparker commented Mar 30, 2022

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

@theadamparker theadamparker added the enhancement New feature or request label Mar 30, 2022
@Jura Jura added this to the 1.0.8 milestone Jun 9, 2022
@theadamparker theadamparker added the design This issue requires design work before development can begin label Jun 9, 2022
@theadamparker
Copy link
Contributor Author

This will require a flexible solution for displaying menu items (particularly more than 4).

@Jura Jura modified the milestones: 1.0.8, 1.0.10 Jun 16, 2022
@Malakeldardiri
Copy link

@Jura Jura assigned Jura and unassigned Malakeldardiri Jul 20, 2022
@Jura Jura reopened this Jul 20, 2022
@Jura
Copy link
Member

Jura commented Jul 20, 2022

Taking over for development

@Jura Jura added Conflicts Conflicts in the PR, cannot merge! extdev outsourcing to external vendor and removed Conflicts Conflicts in the PR, cannot merge! labels Oct 17, 2022
@Jura Jura removed this from the 1.0.10 milestone Nov 7, 2022
paboden added a commit that referenced this issue Feb 1, 2023
Country and Global headers shouldn't change to tablet/mobile layout until it's <1024px

#915

Also includes some css to fix the glitchy mega menu activation. Sets the links to take up the full height of the header so the js that triggers the mega menu has a larger activation area.
@paboden
Copy link
Contributor

paboden commented Feb 1, 2023

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.

Jura added a commit that referenced this issue Feb 1, 2023
…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
@Jura Jura closed this as completed Feb 3, 2023
@Jura
Copy link
Member

Jura commented Mar 2, 2023

@mrelevo , please verify

@Jura Jura reopened this Mar 2, 2023
@mrelevo
Copy link
Contributor

mrelevo commented Mar 2, 2023

verified in Dev

@mrelevo mrelevo closed this as completed Mar 2, 2023
@mrelevo
Copy link
Contributor

mrelevo commented Mar 3, 2023

@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

https://develop.design-system.pages.dev/iframe?args=&id=components-navigation-components-main-navigation-global-header--global-header&viewMode=story

Image

@paboden
Copy link
Contributor

paboden commented Mar 3, 2023

@Jura @mrelevo - Created a new PR to address the logo - #1148

@Jura
Copy link
Member

Jura commented Mar 3, 2023

@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

@Jura
Copy link
Member

Jura commented Mar 6, 2023

@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
Copy link
Contributor

paboden commented Mar 8, 2023

Hi @Jura, This is ready to review #1148

@Jura
Copy link
Member

Jura commented Mar 8, 2023

@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:
image

@theadamparker
Copy link
Contributor Author

@paboden here's another approach at showing the misalignment issue. Note how the language switcher is sitting higher than the menu items.

Screenshot 2023-03-08 at 9 25 42 AM

You can also see this change in Chromatic

@theadamparker
Copy link
Contributor Author

@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

paboden added a commit that referenced this issue Mar 9, 2023
…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
@paboden
Copy link
Contributor

paboden commented Mar 9, 2023

@theadamparker This is ready to review again here - #1148

  • I fixed the progress bar, which just needed the position from top adjusted between mobile and xlarge.
  • I fixed the vertical position of the language switcher to match the position of the menu items in the global header (I also double checked the country header)

Jura added a commit that referenced this issue Mar 9, 2023
#916 -- Fix to logo z-index on large screen to make clickable again
@Jura
Copy link
Member

Jura commented Mar 9, 2023

@mrelevo , over to you for verificaiton

@mrelevo
Copy link
Contributor

mrelevo commented Mar 9, 2023

Hi @paboden, all the issues have been resolved. However, a new one has been introduced. The language selector option now shows below the header

image

See correct behavior here

image

And its fine as a standalone https://develop.design-system.pages.dev/?path=/story/components-ui-components-language-switcher--language-switcher

Jura added a commit that referenced this issue Mar 10, 2023
#916 -- fixed vertical offset language dropdown
@Jura
Copy link
Member

Jura commented Mar 10, 2023

@mrelevo , over to you

@mrelevo
Copy link
Contributor

mrelevo commented Mar 21, 2023

looks good dev

@mrelevo
Copy link
Contributor

mrelevo commented May 9, 2023

verified in storybook and Drupal DEV

@mrelevo mrelevo closed this as completed May 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design This issue requires design work before development can begin enhancement New feature or request extdev outsourcing to external vendor
Projects
None yet
5 participants