Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jun 13, 2025

Fixes an accessibility issue where the DataGrid in the "Select Process Window" popup lacked a proper name for screen readers and assistive technologies.

Problem

When using accessibility tools like "Accessibility Insights for Windows" on the Select Process dialog, the DataGrid element showed as datagrid '' with an empty name, violating the accessibility requirement that "The Name property of a focusable element must not be null."

Solution

Added AutomationProperties.Name="Process selection table" to the DataGrid element in SelectProcess.xaml. This provides a meaningful, descriptive name for screen readers and follows the established pattern used throughout the PerfView codebase for similar controls.

Changes

  • Minimal change: Added one line to /src/PerfView/Dialogs/SelectProcess.xaml
  • Consistent naming: Follows existing patterns like "Events Table", "Multi-Line View", etc.
  • No functional impact: Only improves accessibility, no behavior changes

Testing

  • Verified XAML is well-formed
  • Confirmed change follows established codebase patterns
  • Minimal scope: only 1 file modified with 1 line addition

Fixes #2238.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

…sibility

Co-authored-by: brianrob <6210322+brianrob@users.noreply.github.com>
Copilot AI changed the title [WIP] The Name property of a focusable element must not be null Add AutomationProperties.Name to Process Selection DataGrid for accessibility Jun 13, 2025
Copilot AI requested a review from brianrob June 13, 2025 18:25
@brianrob brianrob marked this pull request as ready for review June 13, 2025 21:13
@brianrob
Copy link
Member

Co-pilot Post-Mortem:

  • The code was easy for copilot to get right.
  • I appreciated that the PR description described the solution and called out that this was done elsewhere for comparison, such that I was able to build reasonable confidence in the fix rather quickly.

@brianrob brianrob merged commit 5cbea4f into main Jun 19, 2025
5 checks passed
@brianrob brianrob deleted the copilot/fix-2238 branch June 19, 2025 22:23
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.

The Name property of a focusable element must not be null

3 participants