fix(a11y): add AT-SPI names for exposure slider and image menu - #523
Conversation
|
Skipping CI for Draft Pull Request. |
Reviewer's guide (collapsed on small PRs)Reviewer's GuideCompletes accessibility naming for the exposure slider and image context menu by setting both Qt object names and accessible names, bringing the reported AT-SPI coverage from 92% to 100%; also updates the image item copyright year range. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Add setObjectName/setAccessibleName for m_slider in ExposureSlider and m_menu in ImageItem so accessibility tools can locate them. 为 ExposureSlider 的 m_slider 和 ImageItem 的 m_menu 补全 setObjectName/setAccessibleName,使无障碍工具可定位控件。 Log: 补全相机AT-SPI控件名称 Influence: 补全2个交互控件的无障碍名称,AT-SPI覆盖率从92%提升至100%。
fdcc21f to
7547610
Compare
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: lzwind, re2zero The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
deepin pr auto review🤖 AI 代码审查报告📊 总体评价
🔍 详细分析1. 语法逻辑 ✅评价: 优秀 ✅ 通过 潜在问题: 建议: [] 2. 代码质量 ✅评价: 优秀 ✅ 通过 潜在问题: 建议: [] 3. 代码性能 ✅评价: 优秀 ✅ 通过 潜在问题: 建议: [] 4. 代码安全 🔒评价: 优秀 ✅ 通过
安全漏洞详情: 建议: [] 💡 改进建议代码示例// 暂无代码示例本报告由 AI 代码审查工具自动生成 |
|
/merge |
AT-SPI 补全 — deepin-camera
补全内容
为以下 2 个交互控件补全
setObjectName()/setAccessibleName():覆盖率对比
验证
分支
fix/at-spi-completion-20260828Summary by Sourcery
Complete AT-SPI naming for the exposure slider and image menu to achieve full accessibility coverage.
Bug Fixes:
Chores: