Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Hidden star banner and back to top buttons are in the keyboard tab order on every page

Đã đóng Phù hợp với người mới
  • Có gì không ổn?
#169 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
86/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
typescript
Lĩnh vực
accessibility, frontend

Hướng nghiên cứu

Đọc các kiểu hiển thị và ẩn trong src/components/StarBanner.astro và src/components/BackToTop.astro, rồi so sánh chúng với menu di động đang đóng trong src/components/Header.astro. Kiểm tra thứ tự tab bằng bàn phím trước và sau khi từng điều khiển trở nên hiển thị hoặc bị ẩn, đồng thời xác nhận các hiệu ứng mờ dần vẫn hoạt động. Công việc hoàn tất khi các điều khiển bị ẩn không thể nhận focus, các điều khiển của banner đang hiển thị vẫn hoạt động và các bước kiểm tra nghiệm thu đều đạt.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

accessibility bug good first issue hacktoberfest

On any page of https://og.codercops.com, Tab past the last footer link ("CODERCOPS"). The next three stops are "Star on GitHub", "Maybe later" and "Close" in the star banner, which isn't on screen, so the focus seems to vanish. On a short page like /preview, the fourth stop is the Back to top button, which is also invisible until you've scrolled 500px. Only then does focus reach the Feedback button.

The banner stays in the tab order even after you've dismissed it for good. And while it's hidden it has aria-hidden="true", so screen reader users can land on controls inside a region that's supposed to be hidden.

Both hide with opacity: 0 and pointer-events: none, which doesn't take anything out of the tab order:

  • src/components/StarBanner.astro: aria-hidden="true" on line 2, the hidden style on lines 94 to 97, and the visible class on lines 100 to 104
  • src/components/BackToTop.astro: the hidden style on lines 48 to 51, and the visible class on lines 54 to 58

The mobile menu already gets this right: it uses visibility: hidden while it's closed (src/components/Header.astro, lines 277 to 291).

What to do

  • Make both unfocusable while they're hidden. The simplest way is CSS: add visibility: hidden to the hidden state and visibility: visible to the visible class, and add visibility to the transition so the fade out still plays. Toggling the inert attribute together with the class works too.
  • Keep the banner's aria-hidden matching what's on screen, as it does now.
  • The back to top script is also part of #161 (scripts that stop working after you follow a link inside the site), so a change that only touches CSS is the easiest to merge alongside it.

Where to look

  • src/components/StarBanner.astro: the markup (lines 2 to 20), hideBanner (lines 47 to 53), where it's shown (lines 57 to 62) and the styles (lines 83 to 104)
  • src/components/BackToTop.astro: toggle (lines 14 to 20) and the styles (lines 32 to 58)
  • src/components/Header.astro: the closed mobile menu (lines 277 to 291), for comparison

Acceptance criteria

  • While the banner is hidden (before it appears, after "Maybe later", or after dismissing it for good), Tab goes from the last footer link to the next control you can see
  • Back to top is in the tab order only while it's visible
  • When the banner is showing, its three controls can be reached with Tab and still work
  • The fade in and out look the same as before
  • npm run check, npm run test and npm run build pass
  • The PR says how you checked the tab order (with the keyboard, or the browser's accessibility tree)

Want to work on this? Comment below and I'll assign it to you. Please take one issue at a time, branch off dev, and open your PR against dev with Closes #169 in the description.

Ngôn ngữ chính
TypeScript
Star
8
Fork
4
Merge trung bình
5 giờ 44 phút
Pull request đã merge (30 ngày)
83

Chuẩn bị môi trường

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.

Issue khác của codercops/ogcops

Tất cả issue của codercops/ogcops

Issue tương tự

Thêm issue về TypeScript

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.