Skip to content

[5.0] Fix color of icon in dark mode#41919

Merged
HLeithner merged 2 commits intojoomla:5.0-devfrom
wilsonge:fix/dark-mode-home-icon
Sep 26, 2023
Merged

[5.0] Fix color of icon in dark mode#41919
HLeithner merged 2 commits intojoomla:5.0-devfrom
wilsonge:fix/dark-mode-home-icon

Conversation

@wilsonge
Copy link
Copy Markdown
Contributor

Pull Request for Issue #41818 .

Summary of Changes

Changes the icon color to reflect text color and removes the near invisible backgroundl

Testing Instructions

Check icon dashboard per original issue in both light and dark mode. The light mode will change color a fraction but shouldn't be significantly noticeable.

Actual result BEFORE applying this Pull Request

menu-dashboard-home

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

@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.0-dev labels Sep 25, 2023
@Quy
Copy link
Copy Markdown
Contributor

Quy commented Sep 25, 2023

I have tested this item ✅ successfully on 7f01e34


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

1 similar comment
@richard67
Copy link
Copy Markdown
Member

I have tested this item ✅ successfully on 7f01e34


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

@richard67
Copy link
Copy Markdown
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Sep 25, 2023
@HLeithner HLeithner merged commit d1e0cef into joomla:5.0-dev Sep 26, 2023
@HLeithner
Copy link
Copy Markdown
Member

Thx

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Sep 26, 2023
@wilsonge wilsonge deleted the fix/dark-mode-home-icon branch September 26, 2023 08:38
@wilsonge wilsonge added this to the Joomla! 5.0 milestone Sep 26, 2023
HLeithner pushed a commit to HLeithner/joomla-cms that referenced this pull request Sep 26, 2023
Co-authored-by: Quy <quy@nomonkeybiz.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NPM Resource Changed This Pull Request can't be tested by Patchtester

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants