? NPM Resource Changed PR-5.0-dev Pending

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
6 Sep 2023

Pull Request for Issue #41571 (Partial).

Summary of Changes

Fixes the save button toolbar double border when editing and also fixes an undefined variable in the tabs element too that invisible but still was a technical bug. bg-color was a bad copy paste from quickicons (where it was correct specifically ther) and should have been body-bg

For the toolbar I’ve been consistent with the original dark mode approach and removed the background just hiding the extra border. But if this causes any unexpected issues I’ll revert to using body-bg there too.

Testing Instructions

Check images from original issue specifically related to the toolbar icon. Should be the same as the Joomla 4.x layout after patch applied

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 - 6 Sep 2023
Category Repository NPM Change
avatar wilsonge wilsonge - open - 6 Sep 2023
avatar wilsonge wilsonge - change - 6 Sep 2023
Status New Pending
avatar wilsonge wilsonge - change - 6 Sep 2023
The description was changed
avatar wilsonge wilsonge - edited - 6 Sep 2023
avatar brianteeman
brianteeman - comment - 6 Sep 2023

:)

image

avatar wilsonge
wilsonge - comment - 6 Sep 2023

That was the view I was testing for this patch :/ on phone again but will drop a screenshot in later to prove I actually did test things ?

avatar wilsonge wilsonge - change - 6 Sep 2023
Labels Added: NPM Resource Changed PR-5.0-dev
avatar wilsonge
wilsonge - comment - 6 Sep 2023

OK I give up. I guess I didn't hit compile after the last time I changed it. Should be fixed.

avatar brianteeman brianteeman - test_item - 6 Sep 2023 - Tested successfully
avatar brianteeman
brianteeman - comment - 6 Sep 2023

I have tested this item ✅ successfully on 725210d


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

avatar Quy Quy - test_item - 7 Sep 2023 - Tested successfully
avatar Quy Quy - alter_testresult - 7 Sep 2023 - brianteeman: Tested successfully
avatar Quy
Quy - comment - 7 Sep 2023

I have tested this item ✅ successfully on 3951f09


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

avatar Quy Quy - change - 7 Sep 2023
Status Pending Ready to Commit
avatar Quy
Quy - comment - 7 Sep 2023

RTC


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

avatar HLeithner HLeithner - change - 7 Sep 2023
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-09-07 08:25:31
Closed_By HLeithner
Labels Added: ?
avatar HLeithner HLeithner - close - 7 Sep 2023
avatar HLeithner HLeithner - merge - 7 Sep 2023

Add a Comment

Login with GitHub to post a comment