RTC NPM Resource Changed bug PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
3 May 2026

Pull Request resolves #47728 .

  • 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

use the var --light for the color of the close X

Testing Instructions

Apply pr and npm i (or use a prebuilt package)
In the media manager open an image for preview and check the close icon

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

image

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 brianteeman brianteeman - open - 3 May 2026
avatar brianteeman brianteeman - change - 3 May 2026
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 3 May 2026
Category Repository NPM Change
avatar brianteeman brianteeman - change - 4 May 2026
Labels Added: NPM Resource Changed PR-5.4-dev
avatar krishnagandhicode krishnagandhicode - test_item - 4 May 2026 - Tested successfully
avatar krishnagandhicode
krishnagandhicode - comment - 4 May 2026

I have tested this item ✅ successfully on 8ce7f16


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

avatar krishnagandhicode
krishnagandhicode - comment - 4 May 2026

I have tested this item ✅ successfully on 8ce7f16


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

avatar chmst chmst - test_item - 4 May 2026 - Tested successfully
avatar chmst
chmst - comment - 4 May 2026

I have tested this item ✅ successfully on 8ce7f16


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

avatar chmst
chmst - comment - 4 May 2026

I have tested this item ✅ successfully on 8ce7f16


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

avatar chmst chmst - change - 4 May 2026
Status Pending Ready to Commit
avatar chmst
chmst - comment - 4 May 2026

RTC


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

avatar chmst
chmst - comment - 4 May 2026

RTC


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

avatar muhme muhme - change - 5 May 2026
Labels Added: RTC
avatar muhme
muhme - comment - 5 May 2026

✅ Final test before merge using JBT

  • Seen grey image close button in grey overlay before PR
  • Applied PR with Patch Tester and npm run build:css
    • Close button is now white and clear visible in light mode
    • Close button still white and clear visible in dark mode
avatar muhme muhme - change - 5 May 2026
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2026-05-05 14:26:00
Closed_By muhme
Labels Added: bug
avatar muhme muhme - close - 5 May 2026
avatar muhme muhme - merge - 5 May 2026
avatar muhme
muhme - comment - 5 May 2026

Thank you very much @brianteeman for your contribution. Thank you @krishnagandhicode for support. Thanks to @krishnagandhicode and @chmst for testing.

avatar brianteeman
brianteeman - comment - 5 May 2026

Thanks

Add a Comment

Login with GitHub to post a comment