NPM Resource Changed PR-5.0-dev Dark Mode Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
7 Oct 2023

Summary of Changes

This feels a bit hacky someone better than me can improve it I am sure.

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

image

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

696e4a9 7 Oct 2023 avatar brianteeman cs
avatar brianteeman brianteeman - open - 7 Oct 2023
avatar brianteeman brianteeman - change - 7 Oct 2023
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 7 Oct 2023
Category Repository NPM Change
avatar HLeithner
HLeithner - comment - 7 Oct 2023

waiting for a review by @wilsonge and a successful test

avatar HLeithner
HLeithner - comment - 7 Oct 2023

@coolcat-creations @MacJoom what's your plan for this, seems similar like the alerts for me?

avatar coolcat-creations
coolcat-creations - comment - 7 Oct 2023

Yes that's also already in the draft

avatar HLeithner
HLeithner - comment - 8 Oct 2023

Thanks then I'm closing this, @MacJoom please keep track of this in #42010

avatar HLeithner HLeithner - close - 8 Oct 2023
avatar HLeithner HLeithner - change - 8 Oct 2023
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2023-10-08 05:00:49
Closed_By HLeithner
Labels Added: NPM Resource Changed PR-5.0-dev Dark Mode
avatar brianteeman
brianteeman - comment - 8 Oct 2023

Sorry I don't understand why this is closed. It is no different to any of the other dark mode fixes that you have merged in the last week. First impressions count and this fixes a bug on the very first page of the admin. In addition it actually works whereas the equivalent in the large pr that will not be merged in 5.0 does not

Add a Comment

Login with GitHub to post a comment