XWIKI-22590: Links in message boxes lack contrast #3604
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Jira URL
https://jira.xwiki.org/browse/XWIKI-22590
Changes
Description
Clarifications
@alert-<type>-bg
instead of@status-<type>-bg
. The alert bg is more of a background, while the status bg is more of a border color.Screenshots & Video
2024-10-25.14-44-57.mp4
The color variation is minimal, and can only be seen on the noColorTheme situation AFAIK.
Executed Tests
Manual tests. Note that I didn't find a way to clear the
noTheme
cache, so I had to use a fresh instance with the messages.less file updated before the first boot to get the expected results.Running
mvn clean install -f xwiki-platform-core/xwiki-platform-flamingo/xwiki-platform-flamingo-skin/xwiki-platform-flamingo-skin-test/xwiki-platform-flamingo-skin-test-docker -Pdocker -Dxwiki.enforcer.skip=true -Dxwiki.test.ui.wcag=true -Dit.test=RenamePageIT#renamePageUsedInMacroContentAndParameters -Dgradle.cache.local.enabled=false -Dgradle.cache.remote.enabled=false
before the changes successfully returned the WCAG violation, while after building the changes withmvn clean install -f xwiki-platform-core/xwiki-platform-flamingo/xwiki-platform-flamingo-skin/xwiki-platform-flamingo-skin-resources
, the contrast violation was not reported anymore.Expected merging strategy