Skip to content

fix: add accessible names to interactive controls across 21 views#868

Merged
laurentiu021 merged 1 commit into
mainfrom
fix/accessibility-names-batch
Jun 12, 2026
Merged

fix: add accessible names to interactive controls across 21 views#868
laurentiu021 merged 1 commit into
mainfrom
fix/accessibility-names-batch

Conversation

@laurentiu021

Copy link
Copy Markdown
Owner

Summary

Adds AutomationProperties.Name to interactive controls across 21 views that lacked
accessible names, so screen readers announce them meaningfully. Generated by a per-view
pass where each view was checked first (a prior accessibility pass — #830/833/835/837/
839/841/843 — already named many controls; those were left untouched and not duplicated).

Scope

~144 names added across: DnsHosts, DiskAnalyzer, DuplicateFile, AppBlocker,
ShortcutCleaner, ContextMenu, BulkInstaller, DeepCleanup, FileShredder, Logs, Ping,
Privacy, ProcessManager, Services, SpeedTest, ThemePopup, Traceroute, Uninstaller,
AppAlerts, AppUpdates, Performance.

Covers: destructive buttons (Delete/Shred/Clear History), icon-only and per-row actions
(named after the row item, e.g. Remove host entry {0}, Add {0} to install list),
toggles, ComboBoxes, search/filter TextBoxes, DataGrids, and theme RadioButtons.

Guarantees

  • Only the AutomationProperties.Name attribute was added — no layout, style,
    binding, value, or any other attribute changed. Zero visual change. (Verified by diff:
    every removed line is the same element re-emitted with the new attribute appended.)
  • Controls that already had names were skipped (not duplicated).
  • Build: main + Tests + IntegrationTests all 0 warnings / 0 errors — every added
    attribute compiles, so the XAML stays well-formed.

fix: → patch release (1.20.20). Protocol C to follow.

@laurentiu021 laurentiu021 merged commit de9638f into main Jun 12, 2026
4 checks passed
@laurentiu021 laurentiu021 deleted the fix/accessibility-names-batch branch June 12, 2026 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant