NPM Resource Changed PR-5.0-dev Dark Mode Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
5 Oct 2023

Summary of Changes

In the changes to support dark mode the color was changed for the titles of the sample data (highlighted below) which unfortunately fails the contrast checks in both light and dark mode.

This PR addresses that.

It is NOT the same as it was in j4 to make the dark mode change automatic. Its really a minor thing but as its on the home page it would be highlighted immediately by anyone reviewing joomla for accessibility

@wilsonge you ok with this change. its neither the same as j4 nor your version of atum for j5

Testing Instructions

Expected result AFTER applying this Pull Request

image

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 brianteeman brianteeman - open - 5 Oct 2023
avatar brianteeman brianteeman - change - 5 Oct 2023
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 5 Oct 2023
Category Repository NPM Change
avatar Quy Quy - test_item - 5 Oct 2023 - Tested successfully
avatar Quy
Quy - comment - 5 Oct 2023

I have tested this item ✅ successfully on 8fdad72


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

avatar HLeithner HLeithner - change - 5 Oct 2023
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-10-05 14:37:45
Closed_By HLeithner
Labels Added: NPM Resource Changed PR-5.0-dev Dark Mode
avatar HLeithner HLeithner - close - 5 Oct 2023
avatar HLeithner HLeithner - merge - 5 Oct 2023

Add a Comment

Login with GitHub to post a comment