Most screens have a single select for ordering eg option=com_users&view=users
but the following have two - one for the direction and one for the column to order by
administrator/index.php?option=com_users&view=groups
administrator/index.php?option=com_users&view=levels
administrator/index.php?option=com_users&view=notes
Category | ⇒ | UI/UX |
Status | New | ⇒ | Confirmed |
I remember discussing this in another PR already.
Personally I would prefer the two select approach for all views. Imho that's easier to use since the lists are shorter.
@andrepereiradasilva do you think you could take a look at this one please
@brianteeman @Bakual @zero-24 @andrepereiradasilva
Since I was involded in testing @andrepereiradasilva's PRs I can say the ordering inconsistency mentioned above is solved with following PRs (note the status):
administrator/index.php?option=com_users&view=groups #9093 = merged
administrator/index.php?option=com_users&view=levels #9092 = RTC
administrator/index.php?option=com_users&view=notes #9091 = RTC
Yes, those PR was exactly to bring some consistency to the search tools across com_users administrator list views.
For other components, there are also already PR for:
So in total 11 RTCs to be merged for having it nice and consistent and not buggy in Beta3 ... or RC ... hope they will make it.
yes but there are some yet inconsistent. Some have bugs, other still have the old filters/search logic.
Example: com_languages overrides view.
Sure ... but I would estimate you have reduced the number of inconsistent and buggy views from some 70% or 80% to some 10% ... and you fixed the most frequently used ones ... from my point of view worth a special note in the release notes.
But thanks for the hint so all can keep in mind that there is still something to be done and help with that by testing (and maybe coding too if you are too busy to do all alone).
Thanks - I forgot I still had to merge the PR when I checked this issue. Thanks for doing all of this - really appreciate it. Another ticket closed - woohoo
Status | Confirmed | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-02-18 10:29:47 |
Closed_By | ⇒ | brianteeman |
#9091 and #9092 meanwhile are merged too, so the things mentioned by this issue here are solved.
Confirmed.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8463.