mi6 / mi6/ic-ui-kit

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

Ouverte
#653 1 commentaire 0 réactions 1 personne assignée Voir sur GitHub

@MI6-806 y travaille déjà.

Depuis le 9/6/2023.

good first issue page-header component
Langage dominant
TypeScript
Étoiles
53
Forks
63
Merge moyen
2 j 4 h
PR mergées (30 j)
15

Description

## 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](https://user-images.githubusercontent.com/117368893/236495128-db1b3293-2500-4533-b74b-970f877f13ae.png)

## 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.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.