Skip to content

Conversation

@skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented May 4, 2023

Fix #37656

If the admin previously uploaded an image, it is not being deleted when the background is set to color only.
Meaning --image-background was being defined anyway
Now it is being defined but we ensure it is set to no, an invalid value, when the bg is deleted

Fix #38061

We were not passing the current main background to the elementColor function, meaning we were not benefiting from the max/min contrast. Now we do. If a value is too bright or too dark, the color will be replaced to ensure proper contrast.

@szaimen
Copy link
Contributor

szaimen commented May 4, 2023

We were not passing the current main background to the elementColor function, meaning we were not benefiting from the max/min contrast. Now we do. If a value is too bright or too dark, the color will be replaced to ensure proper contrast.

Doesnt seem to fix the problem in my testing:
image

@szaimen
Copy link
Contributor

szaimen commented May 4, 2023

If the admin previously uploaded an image, it is not being deleted when the background is set to color only.
Meaning --image-background was being defined anyway
Now it is being defined but we ensure it is set to no, an invalid value, when the bg is deleted

seems to work but when the admin uploads a background image and then clicks on delete (so that only the background color stays) the by the admin uploaded image will still be shown as standard background even though it was removed:
image

@skjnldsv
Copy link
Member Author

skjnldsv commented May 4, 2023

the by the admin uploaded image will still be shown as standard background even though it was removed

Nice catch! Let me fix too

Doesnt seem to fix the problem in my testing:

What should I see in your screenshot?

@skjnldsv skjnldsv changed the title fix(theming): calc primary element from current main background fix(theming): unwanted variables and colours calc fixes May 4, 2023
@szaimen
Copy link
Contributor

szaimen commented May 4, 2023

What should I see in your screenshot?

The buttons should have a more greyish color to better distinguish them from the background even if the primary color is set to black?

@skjnldsv
Copy link
Member Author

skjnldsv commented May 4, 2023

What should I see in your screenshot?

The buttons should have a more greyish color to better distinguish them from the background even if the primary color is set to black?

Well, those buttons uses primary css variables that are not made for elements
image

That needs to be fixed in the vue components then :)

@skjnldsv skjnldsv requested a review from juliusknorr May 4, 2023 11:02
@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 2. developing Work in progress labels May 4, 2023
@skjnldsv skjnldsv merged commit 263a691 into master May 5, 2023
@skjnldsv skjnldsv deleted the fix/theming branch May 5, 2023 11:57
@skjnldsv
Copy link
Member Author

skjnldsv commented May 5, 2023

/backport to stable26

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish bug feature: theming regression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

--color-primary-element has bad contrast with dark primary colors on dark mode [Bug]: Cannot select a solid color background after update to 26

6 participants