RTC NPM Resource Changed PR-5.1-dev Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
26 Mar 2024

Removes the extra border on the sidebar that is producing a pixel shift between light and dark mode.

I can't see the need for this border buy maybe @coolcat-creations sees something I am missing and it should remain

Pull Request for Issue #43159

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 brianteeman brianteeman - open - 26 Mar 2024
avatar brianteeman brianteeman - change - 26 Mar 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 26 Mar 2024
Category Repository NPM Change
avatar Razzo1987 Razzo1987 - test_item - 26 Mar 2024 - Tested successfully
avatar Razzo1987
Razzo1987 - comment - 26 Mar 2024

I have tested this item ✅ successfully on fd8aaf9

now the menu items do not move horizontally when changing from light to dark theme.
They move vertically, but I think it is related to issue fixed with #43138


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

avatar coolcat-creations
coolcat-creations - comment - 26 Mar 2024

The border was added because of the Wish from release management for better contrast between content and sidebar and toolbar and content @LadySolveig

avatar Razzo1987
Razzo1987 - comment - 26 Mar 2024

The border was added because of the Wish from release management for better contrast between content and sidebar and toolbar and content @LadySolveig

In case it could be a border only on the right?

avatar coolcat-creations
coolcat-creations - comment - 26 Mar 2024

The border was added because of the Wish from release management for better contrast between content and sidebar and toolbar and content @LadySolveig

In case it could be a border only on the right?

If I added it surrounding the sidebar it was indeed a mistake

avatar Quy Quy - test_item - 27 Mar 2024 - Tested successfully
avatar Quy
Quy - comment - 27 Mar 2024

I have tested this item ✅ successfully on fd8aaf9


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

avatar Quy Quy - change - 27 Mar 2024
Status Pending Ready to Commit
Labels Added: NPM Resource Changed PR-5.1-dev
avatar Quy
Quy - comment - 27 Mar 2024

RTC


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

avatar brianteeman brianteeman - change - 3 Apr 2024
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2024-04-03 07:28:10
Closed_By brianteeman
Labels Added: RTC
avatar brianteeman brianteeman - close - 3 Apr 2024

Add a Comment

Login with GitHub to post a comment