NPM Resource Changed PR-5.0-dev Pending

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
19 Sep 2023

Summary of Changes

Moves all use of CSS variables in media manager to the _variables.scss file. This makes it easier to find for templates what CSS variables are required for integration (i.e. everything that is var(--template-* now lives in _variables.scss). For maintainers I've tried to comment all the things that are more than just moving variables around.

Testing Instructions

Check media manager renders the same as before. The only color changes is on active actions menu items when selecting an item and it should effectively be not noticeable.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar joomla-cms-bot joomla-cms-bot - change - 19 Sep 2023
Category Repository NPM Change
avatar wilsonge wilsonge - open - 19 Sep 2023
avatar wilsonge wilsonge - change - 19 Sep 2023
Status New Pending
avatar wilsonge wilsonge - change - 19 Sep 2023
Title
Move integration points for media manager into the variables file
[5.0] Move integration points for media manager into the variables file
avatar wilsonge wilsonge - edited - 19 Sep 2023
057687c 19 Sep 2023 avatar wilsonge CS
avatar wilsonge wilsonge - change - 19 Sep 2023
Labels Added: NPM Resource Changed PR-5.0-dev
avatar wilsonge wilsonge - change - 19 Sep 2023
The description was changed
avatar wilsonge wilsonge - edited - 19 Sep 2023
avatar wilsonge wilsonge - change - 19 Sep 2023
The description was changed
avatar wilsonge wilsonge - edited - 19 Sep 2023
avatar HLeithner HLeithner - change - 20 Sep 2023
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-09-20 08:33:32
Closed_By HLeithner
avatar HLeithner HLeithner - close - 20 Sep 2023
avatar HLeithner HLeithner - merge - 20 Sep 2023
avatar HLeithner
HLeithner - comment - 20 Sep 2023

thanks, I think it's better to have variables there then hardcoded colors.

Add a Comment

Login with GitHub to post a comment