NPM Resource Changed PR-5.0-dev Dark Mode Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
6 Oct 2023

Overrides the $pink variable from bootstrap to improve contrast and ensure it meets accessibility contrast standards

Pull Request for Issue #42072 .

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 brianteeman brianteeman - open - 6 Oct 2023
avatar brianteeman brianteeman - change - 6 Oct 2023
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Oct 2023
Category Repository NPM Change
avatar C-Lodder
C-Lodder - comment - 6 Oct 2023

Doesn't work for me. Still ends up being the original colour.

This seems to work though:

$code-color-dark: #d7488b;

avatar C-Lodder
C-Lodder - comment - 6 Oct 2023

That said, #d7488b is still too dark for the #212529 background

avatar brianteeman
brianteeman - comment - 6 Oct 2023

Please test again - not by testing the pink directly

image

avatar C-Lodder C-Lodder - test_item - 6 Oct 2023 - Tested successfully
avatar HLeithner
HLeithner - comment - 6 Oct 2023

waiting for a review by @wilsonge and a successful test

avatar C-Lodder C-Lodder - test_item - 6 Oct 2023 - Not tested
avatar brianteeman brianteeman - change - 6 Oct 2023
Labels Added: NPM Resource Changed PR-5.0-dev
avatar brianteeman
brianteeman - comment - 6 Oct 2023

Changed from $pink to $color-code-dark

image

image

avatar HLeithner
HLeithner - comment - 7 Oct 2023

still waiting for a review by @wilsonge and a successful test

avatar Quy Quy - test_item - 7 Oct 2023 - Tested successfully
avatar Quy
Quy - comment - 7 Oct 2023

I have tested this item ✅ successfully on a81ff8f


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

avatar HLeithner HLeithner - change - 7 Oct 2023
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-10-07 13:22:57
Closed_By HLeithner
Labels Added: Dark Mode
avatar HLeithner HLeithner - close - 7 Oct 2023
avatar HLeithner HLeithner - merge - 7 Oct 2023

Add a Comment

Login with GitHub to post a comment