Closed oleq closed 1 week ago
The update aims to improve accessibility features in CKEditor components, especially for high-contrast mode compatibility. It introduces mixins for media colors, adjusts color handling in UI components, and enhances visibility in various display settings.
Files | Changes |
---|---|
.../imagecaption.css , .../tablecaption.css , .../ckeditor5-widget/widget.css |
Added _mediacolors.css import and new CSS variables for media color handling. |
.../ckeditor5-ui/components/colorgrid/colorgrid.css |
Included mixins for default and forced media colors; updated color tile styles. |
.../ckeditor5-theme-lark/theme/.../placeholder.css |
Introduced mixins to enhance placeholder text visibility in high contrast and Edge on Windows. |
ckeditor5-table/package.json |
Updated dependencies to include @ckeditor/ckeditor5-ui . |
ckeditor5-ui/src/colorgrid/colortileview.ts , ckeditor5-utils/src/env.ts |
Enhanced color handling based on the environment's forced color settings. |
ckeditor5-ui/theme/mixins/_mediacolors.css |
Added new mixins for managing forced colors in media elements. |
Objective | Addressed | Explanation |
---|---|---|
High contrast mode compatibility improvements (Issue #14907) | ✅ | |
Enhancements for color handling in UI elements (Issue #14907) | ✅ | |
Addressing visibility issues in different color modes (Issue #14907) | ✅ | |
Indirect support for color utility functions (Issue #16260) | ❓ | The focus on theme responsiveness and color handling indirectly aligns with the objectives of enhancing color utility functions but does not directly extend them. |
packages/ckeditor5-image/theme/imagecaption.css (2)
`6-6`: Import of `_mediacolors.css` correctly aligns with the PR's objectives to enhance high contrast mode support. --- `25-29`: Media query correctly enhances placeholder visibility in high contrast mode by unsetting styles, aligning with the PR's objectives.
Suggested merge commit message (convention)
Feature (utils): Implemented the
env#isMediaForcedColors
property for forced colors detection (e.g. high contrast mode on Windows). See #14907.Feature (ui): Implemented
ck-media-forced-colors
andck-media-default-colors
mixins for detecting the forced colors (e.g. high contrast mode on Windows) mode. See #14907.Fix (theme-lark): The caret should be visible in a placeholder while in forced colors mode (e.g. high contrast mode on Windows). Improved the look of the placeholders in the forced colors mode. Closes #14907.
Fix (theme-lark): The color grid component should render as a grid of labels in the forced colors mode (e.g. high contrast mode on Windows). Closes #14907.
Summary by CodeRabbit
New Features
Bug Fixes
Refactor
Documentation