Skip to content

Conversation

@brianteeman
Copy link
Contributor

Pull Request for Issue #42986 (comment) .

Summary of Changes

Adds the alert-link class to the string
Removes the color attribute on the link as it was overriding the specific classes for each type of alert

Testing Instructions

You must be testing this with the latest branch!!

Go to System->Update sites and disable any update site
Go to System->Updates and check for updates

Actual result BEFORE applying this Pull Request

Light Mode

image

Dark Mode

image

Expected result AFTER applying this Pull Request

Light Mode

image

Dark Mode

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

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.1-dev labels Mar 11, 2024
@coolcat-creations
Copy link
Contributor

coolcat-creations commented Mar 11, 2024

thats strange, thats actually one of my last commits / fixes yesterday into the Main PR... Did it went lost?

@coolcat-creations
Copy link
Contributor

ok, its confirmed my commit was removed because it did not work so well ;-) I will test yours now

@coolcat-creations
Copy link
Contributor

Contrast Issue:
grafik

@coolcat-creations
Copy link
Contributor

grafik grafik

@coolcat-creations
Copy link
Contributor

In light mode, only the success link looks wrong

grafik

@bembelimen
Copy link
Contributor

Thanks @brianteeman for fixing, I broke that thing and missed it (was too late). So will put it in the 5.1.0 milestone, it should be fir sure merged.

@bembelimen bembelimen added this to the Joomla! 5.1.0 milestone Mar 11, 2024
@ceford
Copy link
Contributor

ceford commented Mar 12, 2024

I can see the link change from plain to underlined. But I see a different colour scheme. I am using a cms clone and did git pull then npm ci; the Status bar shows 5.1.0-beta2-dev; and I use the Firefox Dark/Light mode switcher (on Mac OS). The backgrounds are all (nearly) black. What am I doing wrong?

@coolcat-creations
Copy link
Contributor

I can see the link change from plain to underlined. But I see a different colour scheme. I am using a cms clone and did git pull then npm ci; the Status bar shows 5.1.0-beta2-dev; and I use the Firefox Dark/Light mode switcher (on Mac OS). The backgrounds are all (nearly) black. What am I doing wrong?

best thing is if you download a package here in this PR and install this

@Fedik Fedik added bug Dark Mode a11y Accessibility labels Mar 13, 2024
@LadySolveig LadySolveig merged commit b6ff5d5 into joomla:5.1-dev Mar 18, 2024
@LadySolveig
Copy link
Contributor

Thank you @brianteeman

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

Labels

a11y Accessibility bug Dark Mode Language Change This is for Translators NPM Resource Changed This Pull Request can't be tested by Patchtester

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants