Feature PR-5.0-dev Small PR-5.2-dev Pending

User tests: Successful: Unsuccessful:

avatar JackKellyUK
JackKellyUK
20 Dec 2022

Pull Request for Issue #39426 .

Summary of Changes

Applied fancy select layout to module filters

Testing Instructions

Apply module filters (Filter Options) and check results filter

Actual result BEFORE applying this Pull Request

List of modules filter accordingly

Expected result AFTER applying this Pull Request

List of modules filter accordingly

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

avatar joomla-cms-bot joomla-cms-bot - change - 20 Dec 2022
Category Administration com_modules
avatar JackKellyUK JackKellyUK - open - 20 Dec 2022
avatar JackKellyUK JackKellyUK - change - 20 Dec 2022
Status New Pending
avatar JackKellyUK JackKellyUK - change - 20 Dec 2022
Labels Added: ?
avatar brianteeman
brianteeman - comment - 20 Dec 2022

I have nothing against this change BUT don't we end up with an inconsistent ui with this only for site modules

avatar JackKellyUK
JackKellyUK - comment - 20 Dec 2022

I have nothing against this change BUT don't we end up with an inconsistent ui with this only for site modules

I agree it would be inconsitent, I'm happy to implement across other filters. My only concern is that when there are only 1 or 2 options it seems unnecessary?
image

avatar Fedik
Fedik - comment - 20 Dec 2022

You should not enable it for everything, only when it realy need

avatar JackKellyUK
JackKellyUK - comment - 20 Dec 2022

You should not enable it for everything, only when it realy need

Where do we draw the line? Only applying the layout to some of the filters will lead to inconsistency as @brianteeman mentioned.

avatar crystalenka
crystalenka - comment - 20 Dec 2022

Generally a fancy select type drop-down is appropriate when there are (expected to be) more than 4 options. Fewer than that it doesn't really make sense or provide any meaningful benefit.

avatar Fedik
Fedik - comment - 20 Dec 2022

Enable it for Menu select, that should be enough.
For fields like State, Language etc. it does not make sense to use it.

Only applying the layout to some of the filters will lead to inconsistency

It will be okay. Look for filters in Article, there Tags and Category are fancy-select, a rest is normal select.

avatar dgrammatiko
dgrammatiko - comment - 20 Dec 2022

Also a heads up that a new select element will be landing in 2023, named selectmenu and will probably deprecate the choices.js. So keep in mind that someone will soon have to revert all these changes and just use a native HTML element (yes, it has a polyfill, so once there's a consensus on the attr/props/methods Joomla can start shipping it, follow the openui project on discord for updates). BTW also dropdowns and other things from Bootstrap soon will need 0 JS and could be done with plain HTML/CSS...

avatar JackKellyUK
JackKellyUK - comment - 20 Dec 2022

It will be okay. Look for filters in Article, there Tags and Category are fancy-select, a rest is normal select.

These have multiple set to true, so they don't have the additional search appear, but appreciate your point.

Okay, the layout is now only applied to the Position, Type and Menu Item as these are the core issue.

I still think there is room for discussion around the consistancy of the filters UI in general.

avatar belinifm
belinifm - comment - 11 Apr 2023

Enable it for Menu select, that should be enough. For fields like State, Language etc. it does not make sense to use it.
Only applying the layout to some of the filters will lead to inconsistency
It will be okay. Look for filters in Article, there Tags and Category are fancy-select, a rest is normal select.

I would say more, in addition to the select menu also in the "Select Parent Menu Item". I have a site where the main menu is extensive and has several menu items with submenus.
When I need to edit a submenu, being able to fetch it for filtering helps a lot.
By the way, in general, for those who use a computer/keyboard it is more practical to type than to search on the scroll.

P.S.
Detail, Joomla 3 has "fancy select" in several filters and in my opinion it is better than Joomla 4 in this requirement. By the way, I don't know why and for what, this useful feature was removed from various system filters, , like the one I mentioned before, for example.

avatar HLeithner
HLeithner - comment - 2 May 2023

This pull request has been automatically rebased to 5.0-dev.

avatar HLeithner
HLeithner - comment - 30 Sep 2023

This pull request has been automatically rebased to 5.1-dev.

avatar dorisdreher dorisdreher - test_item - 24 Feb 2024 - Tested successfully
avatar dorisdreher
dorisdreher - comment - 24 Feb 2024

I have tested this item ✅ successfully on fe5dc1e


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

avatar HLeithner
HLeithner - comment - 24 Apr 2024

This pull request has been automatically rebased to 5.2-dev.

avatar HLeithner HLeithner - change - 24 Apr 2024
Title
Module filters to use fancy select layout
[5.2] Module filters to use fancy select layout
avatar HLeithner HLeithner - edited - 24 Apr 2024
avatar obuisard obuisard - change - 24 Aug 2024
Labels Added: Feature PR-5.0-dev Small PR-5.2-dev
Removed: ?
avatar obuisard
obuisard - comment - 24 Aug 2024

I have been looking around and it looks like we have the same kind of problematic in other filters (like the menu item filters).
It also introduces a visual issue for menuitem:

image

There is no 'heading' for the first group of options.
So the fancy select layout should not have a choice__heading layer if there is no content. Or being hidden...

avatar brianteeman
brianteeman - comment - 24 Aug 2024

while we are using a very outdated version of choices.js all this discussion is pretty pointless.

avatar HLeithner
HLeithner - comment - 2 Sep 2024

This pull request has been automatically rebased to 5.3-dev.

avatar HLeithner HLeithner - change - 2 Sep 2024
Title
[5.2] Module filters to use fancy select layout
[5.3] Module filters to use fancy select layout
avatar HLeithner HLeithner - edited - 2 Sep 2024

Add a Comment

Login with GitHub to post a comment