mi6 / mi6/ic-ui-kit

Z-index is not correct for sticky ic-page-header

Offen
#653 1 Kommentar 0 Reaktionen 1 zugewiesene Person Auf GitHub ansehen

@MI6-806 arbeitet bereits daran.

Seit 09.6.2023.

good first issue page-header component
Vorherrschende Sprache
TypeScript
Sterne
53
Forks
63
Ø Merge
2 T. 4 Std.
Gemergte PRs (30 T.)
15

Beschreibung

Summary of the bug

Spotted on the guidance site, the page header is sticky and should be shown above any page content. However, the ic-menu is displayed over it when it is opened and the page is scrolled.

🪜 How to reproduce

  1. Go to https://design.sis.gov.uk/components/select.
  2. Click the select example on the guidance tab so that the menu is open.
  3. Scroll down whilst keeping the menu open.
  4. The menu will appear above the sticky page header.

🧐 Expected behaviour

A sticky page header should cover all page content.

📸 Screenshots or code

Screenshot from 2023-05-05 16-01-41

Additional info

Will maybe need a new design token for a sticky page header z-index. Could then add that to page header when the sticky prop is applied.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.