Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(switch): 修复朗读不可用问题. (#1018) #1096

Closed
wants to merge 3 commits into from

Conversation

byq1213
Copy link
Contributor

@byq1213 byq1213 commented Nov 22, 2022

🤔 这个 PR 的性质是?

  • 日常 bug 修复
  • 新特性提交
  • 文档改进
  • 演示代码改进
  • 组件样式/交互改进
  • CI/CD 改进
  • 重构
  • 代码风格优化
  • 测试用例
  • 分支合并
  • 其他

🔗 相关 Issue

#1018

💡 需求背景和解决方案

📝 更新日志

☑️ 请求合并前的自查清单

⚠️ 请自检并全部勾选全部选项⚠️

  • 文档已补充或无须补充
  • 代码演示已提供或无须提供
  • TypeScript 定义已补充或无须补充
  • Changelog 已提供或无须提供

演示链接:

https://share.weiyun.com/KbBwx0E0

@syxysszyw
Copy link
Collaborator

录屏00:08,”带描述开关“当开关状态是为“关”时,会播报出“关”这个信息,但处于”开“时却不播报;建议:要么”开“和”关“这2个关键词都能播报,要么都省略不播报(能正确传达”已/未勾选“信息即可)
image

@@ -49,6 +49,7 @@
.@{prefix}-switch {
display: flex;
align-items: center;
overflow: hidden;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个样式是解决什么问题呀?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

读取区域会随着里面dot挪动

@syxysszyw
Copy link
Collaborator

现在的录屏没有将整个示例录完,辛苦 @byq1213 提供一个完整的录屏链接

@syxysszyw
Copy link
Collaborator

syxysszyw commented Nov 30, 2022

从语义化的角度来讲,示例这里是不是在t-switch外层包一个label会更标准些?
image

@byq1213
Copy link
Contributor Author

byq1213 commented Nov 30, 2022

完整演示: https://share.weiyun.com/rys6kBFw

@byq1213
Copy link
Contributor Author

byq1213 commented Nov 30, 2022

从语义化的角度来讲,示例这里是不是在t-switch外层包一个label会更标准些? image

完整演示: https://share.weiyun.com/rys6kBFw
语义化变更已处理 1586118

disabled="{{disabled}}"
aria-checked="{{checked}}"
aria-disabled="{{disabled}}"
aria-role="checkbox"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

为什么是在label上设计aria-role=checkbox呀?

@LeeJim LeeJim added the a11y 无障碍访问 label Dec 2, 2022
@byq1213 byq1213 closed this Dec 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y 无障碍访问
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants