? NPM Resource Changed PR-5.0-dev Pending

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
22 Sep 2023

Pull Request for Issue #41861 .

Summary of Changes

Adds CSS overrides for the modals that are used in guided tours so they render in dark mode. I'm honestly not convinced the color contrast for the header (using the --dark-bg-subtle color) gives enough contrast to the main body background. We may need to go with something a touch lighter. But should be an easy fix if someone can suggest something.

@HLeithner this one is more complicated so recommend we get two good tests here

Testing Instructions

Please test a few different tours so you get different sized modals and text.

Actual result BEFORE applying this Pull Request

dark-tour-guide

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 joomla-cms-bot joomla-cms-bot - change - 22 Sep 2023
Category Repository NPM Change
avatar wilsonge wilsonge - open - 22 Sep 2023
avatar wilsonge wilsonge - change - 22 Sep 2023
Status New Pending
avatar wilsonge wilsonge - change - 22 Sep 2023
Title
Dark mode support for guided tours
[5.0] Dark mode support for guided tours
avatar wilsonge wilsonge - edited - 22 Sep 2023
avatar HLeithner
HLeithner - comment - 23 Sep 2023

@obuisard can you test this please?

avatar obuisard obuisard - test_item - 23 Sep 2023 - Tested successfully
avatar obuisard
obuisard - comment - 23 Sep 2023

I have tested this item ✅ successfully on 0eb52fc


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

Looks cool in dark mode...

avatar Quy Quy - test_item - 24 Sep 2023 - Tested successfully
avatar Quy
Quy - comment - 24 Sep 2023

I have tested this item ✅ successfully on 0eb52fc


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

avatar Quy Quy - change - 24 Sep 2023
Status Pending Ready to Commit
Labels Added: NPM Resource Changed PR-5.0-dev
avatar Quy
Quy - comment - 24 Sep 2023

RTC


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

avatar HLeithner HLeithner - close - 24 Sep 2023
avatar HLeithner HLeithner - merge - 24 Sep 2023
avatar HLeithner HLeithner - change - 24 Sep 2023
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-09-24 06:29:38
Closed_By HLeithner
Labels Added: ?
avatar HLeithner
HLeithner - comment - 24 Sep 2023

Thx

Add a Comment

Login with GitHub to post a comment