User tests: Successful: Unsuccessful:
Added a more specific line for padding-inline-end: 10px; into choices.scss. I did not remove the existing one, because I don't know the side effects.
Before the patch your category dropdown in filter Options is squeezed. After applying the patch it uses the whole width.
Please select:
Documentation link for guide.joomla.org:
[x ] No documentation changes for guide.joomla.org needed
Pull Request link for manual.joomla.org:
[x ] No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
| Category | ⇒ | Repository NPM Change |
In 5.4.3 it's the same code for padding inline end and for the padding-right: 100px I did not remove or change this, I only specified the padding inline end deeper so that it's taken into account.
The change in the original version of choices changed the specificity of the CSS definition:
Version 9.1 (the one we load in 5.4.3)
@media (width >= 640px) {
.choices__list--dropdown .choices__item--selectable {
padding-right: 100px;
}
}
Version 11.1 (the one we load in 6.0)
@media (width >= 640px) {
.choices__list--dropdown .choices__item--selectable[data-select-text], .choices__list[aria-expanded] .choices__item--selectable[data-select-text] {
padding-right: 100px;
}
}
And that is why our override
.choices .choices__list--dropdown .choices__item {
padding-inline-end: 10px;
}
doesn't work anymore
The change in the original version of choices changed the specificity of the CSS definition: Version 9.1 (the one we load in 5.4.3) @media (width >= 640px) { .choices__list--dropdown .choices__item--selectable { padding-right: 100px; } } Version 11.1 (the one we load in 6.0) @media (width >= 640px) { .choices__list--dropdown .choices__item--selectable[data-select-text], .choices__list[aria-expanded] .choices__item--selectable[data-select-text] { padding-right: 100px; } } And that is why our override .choices .choices__list--dropdown .choices__item { padding-inline-end: 10px; } doesn't work anymore
Should I then just replace the padding-inline-end: 10px; instead of adding a more specific one?
Add teh specificity in the already existent code instead of adding new one, I think the media query is not necessary.
| Labels |
Added:
NPM Resource Changed
PR-6.1-dev
|
||
I have tested this item 🔴 unsuccessfully on 2267095
@brianteeman increased it, thanks
| Title |
|
||||||
@brianteeman increased it, thanks
Can you please test it again @brianteeman?
I have tested this item ✅ successfully on 6b4191a
Sorry thought I had already retedted
| Labels |
Added:
bug
|
||
Thanks @bembelimen for branch merge.
Thanks @coolcat-creations for fixing.
Thanks @brianteeman for testing and @drmenzelit for personal review.
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2026-02-27 08:54:14 |
| Closed_By | ⇒ | tecpromotion |
@coolcat-creations @tecpromotion Is there a reason why this PR has been made for 6.1-dev despite of its title "... Bugfix ..."?
The question came up here: #47225 (comment)
See also for chosing the right base branch: https://github.com/joomla/joomla-cms?tab=readme-ov-file#which-branch-should-my-pull-request-target
@richard67 it was the correct branch as it was the only branch with the too big 100px margin. However subsequently we can see a similar problem in the 5.4 branch with a too small 10px margin. So two different problems although the checkmark issue is present in both
@richard67 it was the correct branch as it was the only branch with the too big 100px margin. However subsequently we can see a similar problem in the 5.4 branch with a too small 10px margin. So two different problems although the checkmark issue is present in both
Would be nice if someone could make a PR for 5.4-dev for that.
@richard67 done #47244
My concern with this is that the "offending css" has been there for 8 years - so either no one spotted this problem before OR something else has changed which is exposing this