RTC a11y NPM Resource Changed bug PR-6.2-dev Pending

User tests: Successful: Unsuccessful:

avatar Nikhil-sinhaa
Nikhil-sinhaa
24 Oct 2026

Pull Request resolves #47838

  • I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.
    Summary of Changes

adjusted highlight colors for better contrast
Testing Instructions

Enable administrator dark mode

Create multiple versions of an article

Open:
Articles → Versions → Compare

Verify:
    added text is clearly readable
    removed text is clearly readable
    contrast is improved in dark mode
    compare view remains functional in light mode

Actual result BEFORE applying this Pull Request

Screenshot From 2026-10-24 18-34-48

Expected result AFTER applying this Pull Request

Screenshot From 2026-10-24 17-43-21 Screenshot From 2026-10-24 17-43-53 Screenshot From 2026-10-24 18-01-22 Screenshot From 2026-10-24 18-01-34

Link to documentations

Please select:

  • Documentation link for guide.joomla.org:

  • No documentation changes for guide.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar Nikhil-sinhaa Nikhil-sinhaa - open - 24 Oct 2026
avatar Nikhil-sinhaa Nikhil-sinhaa - change - 24 Oct 2026
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 24 Oct 2026
Category ⇒ JavaScript NPM Change
avatar richard67 richard67 - change - 24 Oct 2026
Title
…
used ins and del
[6.2] Fix/47838 dark mode color contrast
avatar richard67 richard67 - edited - 24 Oct 2026
avatar richard67 richard67 - change - 24 Oct 2026
The description was changed
avatar richard67 richard67 - edited - 24 Oct 2026
avatar richard67 richard67 - change - 24 Oct 2026
The description was changed
avatar richard67 richard67 - edited - 24 Oct 2026
avatar chmst chmst - test_item - 25 Oct 2026 - Tested successfully
avatar chmst
chmst - comment - 25 Oct 2026

I have tested this item ✅ successfully on 90aea39


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/48694.

avatar chmst
chmst - comment - 25 Oct 2026

I have tested this item ✅ successfully on 90aea39


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/48694.

avatar exlemor exlemor - test_item - 25 Oct 2026 - Tested successfully
avatar exlemor
exlemor - comment - 25 Oct 2026

I have tested this item ✅ successfully on 90aea39

I have tested this succesfully. Thanks @Nikhil-sinhaa!

One question? if we are fixing this, why fix it to WCAG-AA level and not all of the way to WCAG-AAA level directly? (just a question), that would mean changing #B3B3B3 currently giving a 4.61:1 ratio to #DCDCDC to give it a 7.06:1 ratio.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/48694.

avatar exlemor
exlemor - comment - 25 Oct 2026

I have tested this item ✅ successfully on 90aea39

I have tested this succesfully. Thanks @Nikhil-sinhaa!

One question? if we are fixing this, why fix it to WCAG-AA level and not all of the way to WCAG-AAA level directly? (just a question), that would mean changing #B3B3B3 currently giving a 4.61:1 ratio to #DCDCDC to give it a 7.06:1 ratio.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/48694.

avatar Nikhil-sinhaa
Nikhil-sinhaa - comment - 25 Oct 2026

First of all Thanks you @exlemor @chmst for testing it out
Great question @exlemor!
Two reasons I went with AA rather than AAA:

  1. Joomla's official accessibility target is WCAG AA
Screenshot From 2026-10-25 20-22-59 https://www.joomla.org/accessibility-statement.html

2.In diff views, deleted text is intentionally muted so attention stays on the new additions. Making it AAA-bright would cause removed content to stand out too much and compete for focus.

Still happy to update to #DCDCDC if the team prefers AAA consistency!

avatar chmst chmst - change - 25 Oct 2026
Status Pending ⇒ Ready to Commit
avatar chmst
chmst - comment - 25 Oct 2026

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/48694.

avatar chmst
chmst - comment - 25 Oct 2026

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/48694.

avatar chmst
chmst - comment - 25 Oct 2026

@exlemor As we aim to have the AA compliance, there is no extra benefit with reachin AAA here. So I think we can accept this.

avatar exlemor
exlemor - comment - 25 Oct 2026

@exlemor As we aim to have the AA compliance, there is no extra benefit with reachin AAA here. So I think we can accept this.

Thanks @chmst - you guys know best! :) Thanks for the clarification.

avatar drmenzelit drmenzelit - change - 30 Oct 2026
Labels Added: RTC a11y NPM Resource Changed bug PR-6.2-dev
avatar drmenzelit drmenzelit - change - 30 Oct 2026
Status Ready to Commit ⇒ Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 ⇒ 2026-10-30 11:52:56
Closed_By ⇒ drmenzelit
avatar drmenzelit drmenzelit - close - 30 Oct 2026
avatar drmenzelit drmenzelit - merge - 30 Oct 2026
avatar drmenzelit
drmenzelit - comment - 30 Oct 2026

Thank you

avatar woluweb
woluweb - comment - 30 Oct 2026

Thank you also !

Add a Comment

Login with GitHub to post a comment