-
Notifications
You must be signed in to change notification settings - Fork 667
Material & Fluent themes: ensure compliance with color-contrast requirements according to a11y standards #32162
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
Material & Fluent themes: ensure compliance with color-contrast requirements according to a11y standards #32162
Conversation
1882cb6 to
0bccf15
Compare
0bccf15 to
06012bc
Compare
1543e79 to
37e97a0
Compare
37e97a0 to
f9dd903
Compare
…tabs, contextmenu, editors known issues
28e771e to
7d0b889
Compare
7d0b889 to
9099a51
Compare
…disabled state, gantt
4908b43 to
9849e10
Compare
44991d3 to
cd154d9
Compare
…hko/DevExtreme into 26_1_color_contrast
There was a problem hiding this comment.
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 217 out of 451 changed files in this pull request and generated 4 comments.
Comments suppressed due to low confidence (3)
apps/demos/Demos/Gantt/TaskTemplate/ReactJs/styles.css:1
- Extra space before semicolon. Remove the trailing space for consistency with other CSS rules in the file.
apps/demos/Demos/Gantt/TaskTemplate/React/styles.css:1 - Extra space before semicolon. Remove the trailing space for consistency with other CSS rules in the file.
apps/demos/Demos/Gantt/TaskTemplate/Angular/app/app.component.css:1 - Extra space before semicolon. Remove the trailing space for consistency with other CSS rules in the file.
4560c9a to
e37a226
Compare
There was a problem hiding this comment.
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 217 out of 449 changed files in this pull request and generated 5 comments.
| const a11yCheckConfig = {}; | ||
|
|
||
| await a11yCheck(t, a11yCheckConfig, '#parentContainer'); |
Copilot
AI
Feb 4, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The a11yCheckConfig is now an empty object and is declared inside the test iteration. This could be moved outside the forEach loop as a constant to avoid unnecessary object creation on each iteration.
e37a226 to
3ab2d39
Compare
There was a problem hiding this comment.
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 150 out of 948 changed files in this pull request and generated 2 comments.
2c1b195 to
7c1d370
Compare
There was a problem hiding this comment.
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 150 out of 946 changed files in this pull request and generated 3 comments.
No description provided.