NPM Resource Changed bug PR-6.1-dev Pending

User tests: Successful: Unsuccessful:

avatar coolcat-creations
coolcat-creations
23 Feb 2026
  • I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.

Summary of Changes

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.

Testing Instructions

Before the patch your category dropdown in filter Options is squeezed. After applying the patch it uses the whole width.

Actual result BEFORE applying this Pull Request

grafik

Expected result AFTER applying this Pull Request

grafik

Link to documentations

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

avatar coolcat-creations coolcat-creations - open - 23 Feb 2026
avatar coolcat-creations coolcat-creations - change - 23 Feb 2026
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Feb 2026
Category Repository NPM Change
avatar brianteeman
brianteeman - comment - 23 Feb 2026

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

avatar coolcat-creations
coolcat-creations - comment - 23 Feb 2026

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.

avatar drmenzelit
drmenzelit - comment - 24 Feb 2026

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

avatar coolcat-creations
coolcat-creations - comment - 24 Feb 2026

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?

avatar drmenzelit
drmenzelit - comment - 24 Feb 2026

Add teh specificity in the already existent code instead of adding new one, I think the media query is not necessary.

avatar coolcat-creations coolcat-creations - change - 25 Feb 2026
Labels Added: NPM Resource Changed PR-6.1-dev
avatar brianteeman brianteeman - test_item - 25 Feb 2026 - Tested unsuccessfully
avatar brianteeman
brianteeman - comment - 25 Feb 2026

I have tested this item 🔴 unsuccessfully on 2267095


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

avatar brianteeman
brianteeman - comment - 25 Feb 2026

Doesnt allow for space for the check mark

image
avatar coolcat-creations
coolcat-creations - comment - 25 Feb 2026

@brianteeman increased it, thanks

avatar tecpromotion tecpromotion - change - 26 Feb 2026
Title
Bugfix - Filteroptions Category List padding CSS
[6.1] Bugfix - Filteroptions Category List padding CSS
avatar tecpromotion tecpromotion - edited - 26 Feb 2026
avatar tecpromotion
tecpromotion - comment - 26 Feb 2026

@brianteeman increased it, thanks

Can you please test it again @brianteeman?

avatar brianteeman brianteeman - test_item - 26 Feb 2026 - Tested successfully
avatar brianteeman
brianteeman - comment - 26 Feb 2026

I have tested this item ✅ successfully on 6b4191a

Sorry thought I had already retedted


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

avatar bembelimen bembelimen - change - 27 Feb 2026
Labels Added: bug
avatar tecpromotion
tecpromotion - comment - 27 Feb 2026

Thanks @bembelimen for branch merge.
Thanks @coolcat-creations for fixing.
Thanks @brianteeman for testing and @drmenzelit for personal review.

avatar tecpromotion tecpromotion - close - 27 Feb 2026
avatar tecpromotion tecpromotion - merge - 27 Feb 2026
avatar tecpromotion tecpromotion - change - 27 Feb 2026
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2026-02-27 08:54:14
Closed_By tecpromotion
avatar richard67
richard67 - comment - 27 Feb 2026

@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

avatar brianteeman
brianteeman - comment - 27 Feb 2026

@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

avatar richard67
richard67 - comment - 27 Feb 2026

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

avatar brianteeman
brianteeman - comment - 27 Feb 2026

Add a Comment

Login with GitHub to post a comment