NPM Resource Changed PR-5.0-dev Pending

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
25 Sep 2023

Pull Request for Issue #41802.

Summary of Changes

Fixes the styling of the create module view in dark mode. Removes the card body class which as far as I can tell is redundant because we aren't using the parent card class but it does override the color of the text.

This is now explicitly sharing the same colors as the quickicons in cpanel (although the hover effects are slightly different as the plus icon isn't a separate action from the main body of text)

Testing Instructions

Create a module in light mode. There shouldn't be a significant change (there will be very minor color changes). Create it in dark mode and it looks like the below image

Actual result BEFORE applying this Pull Request

modules

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

avatar joomla-cms-bot joomla-cms-bot - change - 25 Sep 2023
Category Administration com_modules Repository NPM Change
avatar wilsonge wilsonge - open - 25 Sep 2023
avatar wilsonge wilsonge - change - 25 Sep 2023
Status New Pending
avatar Quy Quy - test_item - 25 Sep 2023 - Tested successfully
avatar Quy
Quy - comment - 25 Sep 2023

I have tested this item ✅ successfully on 210188d


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

avatar HLeithner HLeithner - close - 26 Sep 2023
avatar HLeithner HLeithner - merge - 26 Sep 2023
avatar HLeithner HLeithner - change - 26 Sep 2023
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-09-26 14:07:57
Closed_By HLeithner
Labels Added: NPM Resource Changed PR-5.0-dev
avatar brianteeman
brianteeman - comment - 26 Sep 2023

The same changes will need to be made #41871 if anyone answers my request for help so it can be merged

avatar wilsonge
wilsonge - comment - 26 Sep 2023

Reached out to you on mattermost

avatar brianteeman
brianteeman - comment - 26 Sep 2023

See #41943 as the removal of the class from the markup broke the css

Add a Comment

Login with GitHub to post a comment