RTC NPM Resource Changed bug PR-5.1-dev Dark Mode Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
12 Mar 2024

Summary of Changes

add css and change to [data-bs-theme-dark]

Actual result BEFORE applying this Pull Request

image

image

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 - 12 Mar 2024
avatar brianteeman brianteeman - change - 12 Mar 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 12 Mar 2024
Category Repository NPM Change
avatar brianteeman brianteeman - change - 12 Mar 2024
Title
[5.1] installer drag and drop
[5.1] DARK MODE installer drag and drop
avatar brianteeman brianteeman - edited - 12 Mar 2024
avatar brianteeman brianteeman - change - 13 Mar 2024
Labels Added: NPM Resource Changed PR-5.1-dev
avatar Fedik
Fedik - comment - 13 Mar 2024

I think use of data-color-scheme would be preferable, in non BS files.

avatar brianteeman
brianteeman - comment - 13 Mar 2024

i would but it doesnt work

avatar Fedik
Fedik - comment - 13 Mar 2024

Okay then :)

avatar brianteeman
brianteeman - comment - 13 Mar 2024

maybe it was me and someone else can get it to work

avatar Fedik
Fedik - comment - 14 Mar 2024

@brianteeman I made a quick check and data-color-scheme works to me:

[data-color-scheme=dark] #dragarea.hover #upload-icon,
[data-color-scheme=dark] #dragarea p.lead {
  color: #eee;
}

[data-color-scheme=dark] #dragarea.hover {
  background-color: #666;
  border-color: #eee;
}
avatar brianteeman
brianteeman - comment - 14 Mar 2024

I will take another look. Probably did something daft

avatar brianteeman brianteeman - change - 14 Mar 2024
Labels Added: bug Dark Mode
avatar brianteeman
brianteeman - comment - 14 Mar 2024

@Fedik updated as requested

avatar Fedik Fedik - test_item - 14 Mar 2024 - Tested successfully
avatar Fedik
Fedik - comment - 14 Mar 2024

I have tested this item ✅ successfully on 9d48656


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

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

I have tested this item ✅ successfully on 9d48656


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

avatar Quy Quy - change - 14 Mar 2024
Status Pending Ready to Commit
avatar Quy
Quy - comment - 14 Mar 2024

RTC


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

avatar Quy Quy - change - 14 Mar 2024
Labels Added: RTC
avatar LadySolveig LadySolveig - change - 14 Mar 2024
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-03-14 17:22:19
Closed_By LadySolveig
avatar LadySolveig LadySolveig - close - 14 Mar 2024
avatar LadySolveig LadySolveig - merge - 14 Mar 2024

Add a Comment

Login with GitHub to post a comment