mi6 / mi6/ic-ui-kit

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

オープン
#653 コメント 1 件 リアクション 0 件 担当者 1 名 GitHub で見る

@MI6-806 がすでに取り組んでいます。

2023年6月9日 から。

good first issue page-header component
主要言語
TypeScript
スター
53
フォーク
63
平均マージ
2日 4時間
マージ済み PR(30日)
15

説明

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

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。