Language Change NPM Resource Changed PR-5.1-dev Failure

User tests: Successful: Unsuccessful:

avatar MacJoom
MacJoom
4 Feb 2024

Pull Request for Issue # .

This is the rebase of #42010 for Joomla 5.1

Summary of Changes

Dark mode colors

Testing Instructions

Turn on dark mode in your browser or O/S

Actual result BEFORE applying this Pull Request

old colors

Expected result AFTER applying this Pull Request

new colors

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

5b11b22 2 Oct 2023 avatar MacJoom cs
a8cab5d 6 Oct 2023 avatar MacJoom cs
avatar MacJoom MacJoom - open - 4 Feb 2024
avatar MacJoom MacJoom - change - 4 Feb 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 4 Feb 2024
Category Administration com_associations com_banners com_categories com_content com_finder com_installer com_languages com_tags com_templates com_users com_workflow Language & Strings Modules Templates (admin) JavaScript Repository NPM Change
avatar fgsw
fgsw - comment - 5 Feb 2024

In light mode 'Clear'-Button not visible:

image

avatar Kostelano
Kostelano - comment - 5 Feb 2024

Is this the intended color? The light version has a different color matching.

Screenshot_1

Menu --> Privacy Dashboard, violation for both themes.

Screenshot_2

Different color separator.

Screenshot_3

Wherever the “loading” indicator of something is used, the Joomla icon looks like this. I’m not sure if this is related to this PR, but I’ll report it anyway.

Screenshot_4

avatar Kostelano
Kostelano - comment - 5 Feb 2024

Fields with the disabled="disabled" attribute are not highlighted in the dark theme.

Screenshot_1

avatar bembelimen bembelimen - change - 8 Feb 2024
Labels Added: Language Change NPM Resource Changed PR-5.1-dev
avatar dorisdreher dorisdreher - test_item - 24 Feb 2024 - Tested successfully
avatar dorisdreher
dorisdreher - comment - 24 Feb 2024

I have tested this item ✅ successfully on e5fc19a


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

avatar crommie crommie - test_item - 24 Feb 2024 - Tested successfully
avatar crommie
crommie - comment - 24 Feb 2024

I have tested this item ✅ successfully on e5fc19a

Small changes in colors


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

avatar crimle crimle - test_item - 24 Feb 2024 - Tested unsuccessfully
avatar crimle
crimle - comment - 24 Feb 2024

I have tested this item ? unsuccessfully on e5fc19a

Switching Firefox to dark mode immediately changes Joomla to dark mode too. With or without this Patch


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

avatar Nuyonuyonoina
Nuyonuyonoina - comment - 24 Feb 2024

Colors doesn't change after applying patch.

avatar eddiekonczal eddiekonczal - test_item - 24 Feb 2024 - Tested unsuccessfully
avatar eddiekonczal
eddiekonczal - comment - 24 Feb 2024

I have tested this item ? unsuccessfully on e5fc19a

Cannot reproduce problem before applying patch


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

avatar korenevskiy
korenevskiy - comment - 26 Feb 2024

изображение
I dragged the installation extension with the mouse. And I saw a white standby setup screen.
Quite obviously there should be a dark screen.

Probably there should be a black translucent one here

avatar Quy
Quy - comment - 27 Feb 2024

Wherever the “loading” indicator of something is used, the Joomla icon looks like this. I’m not sure if this is related to this PR, but I’ll report it anyway.

@Kostelano See #42903

avatar korenevskiy
korenevskiy - comment - 29 Feb 2024

new:
изображение
old:
изображение

Colleagues, I understand that a lot of work has already been done, it turned out to be many times more work than expected. And a huge thank @MacJoom .

I want to suggest making the links a little lighter.
The old color is annoying because they differ too much in color from the main text.
I'll give you an example. For example, if we have a text that is bright green and bright red, then when working with the text, we try to relax so that we focus only on the idea of the text. So the eyes focus only on the bright red color, or on the bright green color. Constantly switching the focus of attention from colors is annoying.
There is a similar situation here, the eyes have to change the focus of the text color when reading.
I'm not insisting. But when using the site, you will eventually feel a mild strain in your eyes.
It resembles the tension of text with a translucent color.
Maybe we'll put this amendment in the next PR?

Focus on the white text and then on the links, you will feel that the white and light blue text are perceived as one. And dark blue links are perceived separately, try to check the focus of attention on the text with links.

Example blue link color: #6fbfdb

avatar coolcat-creations
coolcat-creations - comment - 2 Mar 2024

new: изображение old: изображение

Colleagues, I understand that a lot of work has already been done, it turned out to be many times more work than expected. And a huge thank @MacJoom .

I want to suggest making the links a little lighter. The old color is annoying because they differ too much in color from the main text. I'll give you an example. For example, if we have a text that is bright green and bright red, then when working with the text, we try to relax so that we focus only on the idea of the text. So the eyes focus only on the bright red color, or on the bright green color. Constantly switching the focus of attention from colors is annoying. There is a similar situation here, the eyes have to change the focus of the text color when reading. I'm not insisting. But when using the site, you will eventually feel a mild strain in your eyes. It resembles the tension of text with a translucent color. Maybe we'll put this amendment in the next PR?

Focus on the white text and then on the links, you will feel that the white and light blue text are perceived as one. And dark blue links are perceived separately, try to check the focus of attention on the text with links.

Example blue link color: #6fbfdb

The color chosen for the links is selected to work for white on linkcolor and on linkcolor on white. So we have only one variable. Another color means way more work and is prohibiting this PR to be ready any time soon

avatar korenevskiy
korenevskiy - comment - 2 Mar 2024

@coolcat-creations
Do you want to say that in a dark theme and in a white theme, the color of the links will be the same, without changes?

avatar coolcat-creations
coolcat-creations - comment - 2 Mar 2024

@coolcat-creations

Do you want to say that in a dark theme and in a white theme, the color of the links will be the same, without changes?

No I was talking only about the dark theme

avatar korenevskiy
korenevskiy - comment - 4 Mar 2024

No I was talking only about the dark theme

I didn't know that the dark theme has a white background for the text.
Sorry. Alas.
How deep is the edit to make. I would suggest that you definitely plan to add it after making this PR.
Have the participants already discussed the plans for a new template based on cassiopeia?
I hope you will agree that my suggestion of link colors is important.

avatar coolcat-creations
coolcat-creations - comment - 4 Mar 2024

Sorry that was a mistake in my text, i meant link color on dark and white on link color. Cassiopeia is totally unrelated to the backend template.

avatar coolcat-creations
coolcat-creations - comment - 4 Mar 2024

@LadySolveig i think all the outstanding issues are fixed can we merge and create afterwards separate PRs if something comes up?

avatar brianteeman
brianteeman - comment - 4 Mar 2024

@coolcat-creations @MacJoom Please remove the block of testing code in the template

image

avatar coolcat-creations
coolcat-creations - comment - 4 Mar 2024

Already reverted actually

avatar brianteeman
brianteeman - comment - 4 Mar 2024

image

image

avatar brianteeman
brianteeman - comment - 4 Mar 2024

Already reverted actually

Not sure where its reverted because it is still present in this pr

avatar coolcat-creations
coolcat-creations - comment - 4 Mar 2024

Already reverted actually

Not sure where its reverted because it is still present in this pr

I only can make PRs to the other PR I cant do a separate PR because this is not merged yet, so we always need to wait until its resolved in the main. Thats also the reason why I am begging to merge this so fixes can be done in smaller PRs

avatar brianteeman
brianteeman - comment - 4 Mar 2024

I only can make PRs to the other PR

Of course you can - you make it to MacJoom:dark-mode-5.1-dev

avatar coolcat-creations
coolcat-creations - comment - 4 Mar 2024

Which I did, thats what I am saying :-)

avatar brianteeman
brianteeman - comment - 4 Mar 2024

so just ask @MacJoom to merge it or to gicve you access on his repo to merge it. That shouldnt be a reason to merge a pr here that is known not to be ready

avatar brianteeman
brianteeman - comment - 4 Mar 2024

image

The new template screenshots in this PR are not used as they are jpg but the code is looking for png

image

avatar coolcat-creations
coolcat-creations - comment - 4 Mar 2024

I did already and he did, I only told you that Its reverted already. Still standing by my point that it would be easier if we have smaller PRs this is hard to work on

avatar Quy
Quy - comment - 4 Mar 2024

Please fix file conflict.

avatar bembelimen bembelimen - change - 9 Mar 2024
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2024-03-09 08:40:41
Closed_By bembelimen
avatar bembelimen
bembelimen - comment - 9 Mar 2024

Closed in favour of #42986

avatar bembelimen bembelimen - close - 9 Mar 2024

Add a Comment

Login with GitHub to post a comment