Carousel buttons have the wrong or missing screen-reader label
未关闭
适合新手
还没有人认领这个 Issue。
bug
- 主要语言
- Ruby
- 星标
- 1k
- 派生
- 67
- 平均合并
- 3 天 12 小时
- 30 天内合并 PR
- 14
描述
- Where:
carousel_previous.rbrendersspan.sr-only "Next slide";
carousel_next.rbrenders nosr-onlyspan at all. Snapshot
carousel/verticalline 17. - Effect: a screen reader announces the previous button as "Next slide"
and the next button as unlabeled. - Fix: "Previous slide" on the previous button, add "Next slide" to the
next button; re-record (on the 2.0 line)carousel/*.
Surfaced by the golden HTML suite (PR #536), which pins what 1.6 renders today — this defect included. Per maintainer decision (2026-09-19) main stays as is: this is addressed on the 2.0 line, and the golden snapshot that pins it is re-recorded there with the diff reviewed.
Source: design/v2/follow-up-issues.md.
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
从 carousel_previous.rb 和 carousel_next.rb 开始,然后检查第 17 行的 snapshot carousel/vertical,以及 2.0 线上的其他 carousel/* golden snapshot。运行 golden HTML suite 并重新记录 carousel/*;当上一个和下一个按钮具有正确的屏幕阅读器标签,并且审查过的 snapshot diff 反映了这些标签时,即表示完成。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- ruby
- 领域
- accessibility
- Issue 类型
- 缺陷
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 活跃度
- 活跃
- 描述清晰度
- 描述清楚
- 新手友好度
- 72/100