Skip to content

A11Y: fix testing#34027

Open
dmlvr wants to merge 3 commits into
DevExpress:26_1from
dmlvr:26_1_a11y_testing
Open

A11Y: fix testing#34027
dmlvr wants to merge 3 commits into
DevExpress:26_1from
dmlvr:26_1_a11y_testing

Conversation

@dmlvr

@dmlvr dmlvr commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@dmlvr dmlvr self-assigned this Jun 16, 2026
Copilot AI review requested due to automatic review settings June 16, 2026 15:30
@dmlvr dmlvr added the 26_1 label Jun 16, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the demos accessibility (axe) TestCafe runner to reduce false-positive failures after axe-core changes, keeping CI accessibility checks passing for affected demos/themes.

Changes:

  • Disables the scrollable-region-focusable axe rule for demos accessibility runs.
  • Extends the Material/Fluent color-contrast ignore list to include multiple Diagram demos.
  • Removes the now-redundant per-demo suppression of scrollable-region-focusable for DataGrid-RemoteCRUDOperations.

Comment thread apps/demos/testing/common.test.ts
Copilot AI review requested due to automatic review settings June 16, 2026 16:20

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comment on lines +65 to +68
const scrollableFalsePositiveComponents = [
'Chat', 'Diagram', 'Gantt', 'PivotGrid', 'Scheduler',
'ScrollView', 'Sortable', 'TileView', 'TreeView',
];
'CardView-FieldTemplate',
// False positive: contrast rules do not apply to read-only editors on the custom option panel background
'VectorMap-DynamicViewport',
// False positive: Diagram uses custom shape colors that do not meet contrast requirements
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants