No Code Attached Yet bug Dark Mode
avatar wilsonge
wilsonge
17 Sep 2023

We have two places in core where we make users choose options:

  1. The link color. This is not AA a11y compatible in Dark Mode - bootstrap itself has two separate colors for this. Do we want to give users the options to have dark and light mode
  2. The template 'special' color. This text is basically invisible in dark mode

41409-permissions

Again do we want to have separate colors for this or not? It's presumably going to be hard to have a color that gives A11y compliance in both light and dark modes.

avatar wilsonge wilsonge - open - 17 Sep 2023
avatar joomla-cms-bot joomla-cms-bot - change - 17 Sep 2023
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 17 Sep 2023
avatar HLeithner
HLeithner - comment - 17 Sep 2023

Different color make sense i think

avatar dgrammatiko
dgrammatiko - comment - 17 Sep 2023

@HLeithner @wilsonge I'm not convinced that there such a need for another colour. Here's my take with Muta
Screenshot 2023-09-17 at 21 03 37

Screenshot 2023-09-17 at 21 03 27
avatar wilsonge wilsonge - change - 27 Sep 2023
Labels Added: Release Blocker
avatar wilsonge wilsonge - labeled - 27 Sep 2023
avatar wilsonge
wilsonge - comment - 27 Sep 2023

I’m marking this one as a release blocker as whether we merge the color changes or not these template params need dealing with

avatar dgrammatiko
dgrammatiko - comment - 27 Sep 2023

these template params need dealing with

There's still a need to deal with the -rgb vars for these colours. In muta I'm doing it directly in the js of the field but for atum (due to b/c) there should be some update fn in the PHP to introduce these values in the db table styles. For ref: https://github.com/dgrammatiko/muta/blob/d6eb3f2f5d12700ae4d461f6fbe5d3cacf266684/media_source/templates/administrator/muta/js/dgcolor-field.mjs#L121-L130

avatar Quy Quy - change - 6 Oct 2023
Labels Added: Dark Mode
avatar Quy Quy - labeled - 6 Oct 2023
avatar Hackwar Hackwar - change - 27 Mar 2024
Labels Added: bug
avatar Hackwar Hackwar - labeled - 27 Mar 2024
avatar Quy Quy - change - 29 Mar 2024
Labels Removed: Release Blocker
avatar Quy Quy - unlabeled - 29 Mar 2024
avatar brianteeman
brianteeman - comment - 7 May 2024

as far as I can tell btn-secondary no longer uses the special colour

avatar brianteeman
brianteeman - comment - 22 Jul 2024

Looks like this can be closed

avatar wilsonge wilsonge - change - 24 Jul 2024
Status New Closed
Closed_Date 0000-00-00 00:00:00 2024-07-24 16:00:43
Closed_By wilsonge
avatar wilsonge wilsonge - close - 24 Jul 2024

Add a Comment

Login with GitHub to post a comment