patternfly / patternfly/patternfly-react
cleanup - DataListAction - id, aria-label and aria-labelledby are not currently in use but are required
未关闭
还没有人认领这个 Issue。
Breaking change :boom:
PF Team
Pinned
- 主要语言
- TypeScript
- 星标
- 862
- 派生
- 392
- 平均合并
- 4 天 8 小时
- 30 天内合并 PR
- 9
描述
id can probably remain (and if so should be made non-required) but aria-label and aria-labelledby should be removed because they don't usually exist on a div without a role. These props were a holdover from when we were using a Button internally opposed to children.
Jira Issue: PF-107
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
定位 DataListAction 组件,并检查其 prop 定义和使用情况。确认 id 是否仍然需要以及是否可以设为可选,然后移除未使用的 aria-label 和 aria-labelledby props。完成标准是公共 props 和相关引用一致地反映该组件当前基于 children 的实现。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- react, typescript
- 领域
- accessibility, frontend
- Issue 类型
- 重构
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 活跃度
- 停滞
- 描述清晰度
- 基本清楚
- 新手友好度
- 45/100