Announce moves to screen readers and make the board one tab stop

Đang mở
  • Có gì không ổn?
#56 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

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

Đánh giá

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

Hướng nghiên cứu

Read js/board.js:23-24 and js/board.js:99-136 for square creation and keyboard focus, then trace move handling in js/game.js:255-258 and the live region in index.html:25-27. Review css/board.css for animation rules and run npm test and npm run lint. Done when the board is one Tab stop, screen readers announce moves and check, and reduced motion is respected.

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

Mô tả

enhancement help wanted severity: medium

Summary

Every square is in the tab order, so a keyboard user presses Tab 64 times to get past the board. The only live region announces whose turn it is, never the move that was played or that the king is in check.

Where

  • js/board.js:23-24 (role="gridcell", tabindex="0" on every square; no role="row" parents)
  • js/board.js:99-136 (keyboard handler and focusSquare)
  • index.html:25-27 (#game-status live region), js/game.js:255-258
  • CSS has no prefers-reduced-motion rule

Proposal

  • Roving tabindex: one square has tabindex="0", the rest -1, updated in focusSquare.
  • Wrap squares in role="row" containers.
  • Announce the move in the live region, for example "White knight to f3. Check. Black's turn", built from the existing move object.
  • Give legal targets of the selected piece a non-visual marker (label suffix or aria-describedby).
  • Add a prefers-reduced-motion rule.

Who benefits

Keyboard and screen reader users. The README lists keyboard navigation and ARIA support as features.

Acceptance criteria

  • One Tab press moves focus past the board
  • A screen reader announces each move and check
  • Animations are reduced when the user asks for reduced motion

Getting started

  • Relevant files: js/board.js, js/game.js, index.html, css/board.css
  • How to run tests: npm install, then npm test and npm run lint
  • Related docs: docs/ARCHITECTURE.md, .github/CONTRIBUTING.md

Filed by repo-audit/full-audit at 8a2c86c. Severity: medium. Effort: M. audit-key: enhancement|js/board.js|createBoard.tabindex

Ngôn ngữ chính
JavaScript
Star
4
Fork
2
Không có pull request nào được merge trong 30 ngày

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 TMHSDigital/Static_Chess

Tất cả issue của TMHSDigital/Static_Chess

Issue tương tự

Thêm issue về JavaScript

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.