Skip to content

[5.3] Choices force background#45842

Merged
richard67 merged 2 commits intojoomla:5.3-devfrom
brianteeman:choices
Aug 7, 2025
Merged

[5.3] Choices force background#45842
richard67 merged 2 commits intojoomla:5.3-devfrom
brianteeman:choices

Conversation

@brianteeman
Copy link
Copy Markdown
Contributor

Pull Request for Issue #45765 .

Summary of Changes

Force the choices background to be used even when mobile otherwise the selected choice is invisible

This is caused by a conflicting css rule that is triggered on mobile and gets priority

[aria-selected=true] {
background-color: var(--body-bg);
}

Someone with better css skills than me might have an alternative/better solution

Testing Instructions

Add a tag to an article and check that when on mobile (small screen) you can still see the selected tag

this is an scss change so you will either need to npm run build:css or use a prebuilt package

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

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

Signed-off-by: BrianTeeman <brian@teeman.net>
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.3-dev labels Aug 5, 2025
@richard67 richard67 added the bug label Aug 5, 2025
@exlemor
Copy link
Copy Markdown

exlemor commented Aug 6, 2025

I have tested this item ✅ successfully on dd0875f

I have successfully test this. Nice catch @fuscage and nice fix @brianteeman.


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

@alikon
Copy link
Copy Markdown
Contributor

alikon commented Aug 6, 2025

I have tested this item ✅ successfully on dd0875f


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

@joomla-cms-bot joomla-cms-bot removed NPM Resource Changed This Pull Request can't be tested by Patchtester bug PR-5.3-dev labels Aug 6, 2025
@alikon
Copy link
Copy Markdown
Contributor

alikon commented Aug 6, 2025

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Aug 6, 2025
@alikon alikon added NPM Resource Changed This Pull Request can't be tested by Patchtester bug PR-5.3-dev labels Aug 6, 2025
@richard67 richard67 merged commit ca1483f into joomla:5.3-dev Aug 7, 2025
39 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Aug 7, 2025
@richard67
Copy link
Copy Markdown
Member

Thanks all (author and testers).

@richard67 richard67 added this to the Joomla! 5.3.3 milestone Aug 7, 2025
@brianteeman
Copy link
Copy Markdown
Contributor Author

thanks

@brianteeman brianteeman deleted the choices branch August 7, 2025 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 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.

6 participants