User tests: Successful: Unsuccessful:
Renamed everything what has in common with the bg-dark color to Primary colour and the special color to secondary color. Moved the secondary color into the first column ob the backend settings.
Apply the patch
npm run build:css
The color names in the setting should be now primary and secondary color and everything should work just like before the patch.
Every modern framework uses the terms Primary Color and Secondary color and for increased usability I suggest to change the naming in Joomla.
Also it does not fix a color to have a specific brightness. Since the dark sidebar is gone and we don't need anymore to calculate all the shades of the sidebar for a11y another step would be also to remove the hue slider and replace it by a colorfield where the user is free to chose the color.
|Category||⇒||Administration Language & Strings Templates (admin)|
The color names in the setting should be now primary and secondary color
Sensible change - thanks
Why not use
TPL_ATUM_COLORS_SETTINGS_PRIMARY_COLOUR_LABEL instead of the long sentence for a label?
Why not use
TPL_ATUM_COLORS_SETTINGS_PRIMARY_COLOUR_LABELinstead of the long sentence for a label?
Actually as said above I would rather replace the hue field by really a primary colour field so this would be solved with another PR. The hue made sense when we calculated colors for the sidebar - now quite not anymore I think.