mi6 / mi6/ic-ui-kit

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

Đang mở
#653 1 bình luận 0 reaction 1 người được giao Xem trên GitHub

@MI6-806 đang làm issue này rồi.

Từ ngày 9/6/2023.

good first issue page-header component
Ngôn ngữ chính
TypeScript
Star
53
Fork
63
Merge trung bình
2 ngày 4 giờ
Pull request đã merge (30 ngày)
15

Mô tả

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.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.