?
avatar brianteeman
brianteeman
17 Nov 2015

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

avatar brianteeman brianteeman - open - 17 Nov 2015
avatar zero-24 zero-24 - change - 25 Nov 2015
Category UI/UX
avatar zero-24 zero-24 - change - 25 Nov 2015
Status New Confirmed
avatar zero-24
zero-24 - comment - 25 Nov 2015

Confirmed.


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

avatar Bakual
Bakual - comment - 25 Nov 2015

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.

avatar brianteeman
brianteeman - comment - 18 Feb 2016

@andrepereiradasilva do you think you could take a look at this one please


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

avatar richard67
richard67 - comment - 18 Feb 2016

@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

avatar andrepereiradasilva
andrepereiradasilva - comment - 18 Feb 2016

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:

  • [com_checkin] checkin view #8854 = RTC
  • [com_cache] cache view #9121 - RTC
  • [com_menus] menus view #8842 = Merged
  • [com_tags] tags view #9094 = Merged
  • [com_installer] update view #9118 = RTC
  • [com_installer] manage view #9106 = Merged | #9139 = RTC
  • [com_installer] discover view #9114 = RTC
  • [com_installer] languages view #9089 = RTC
  • [com_installer] updatesites view #9107 = Merged | #9139 = RTC
  • [com_templates] styles and templates views #9090 = RTC
  • [com_languages] installed view #9101 = RTC
  • [com_languages] languages view #9099 = Merged
avatar richard67
richard67 - comment - 18 Feb 2016

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.

avatar andrepereiradasilva
andrepereiradasilva - comment - 18 Feb 2016

yes but there are some yet inconsistent. Some have bugs, other still have the old filters/search logic.
Example: com_languages overrides view.

avatar richard67
richard67 - comment - 18 Feb 2016

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

avatar andrepereiradasilva
andrepereiradasilva - comment - 18 Feb 2016

Update:

  • [com_users] groups view #9093 = Merged
  • [com_users] levels view #9092 = Merged
  • [com_users] notes view #9091 = Merged

So this issue can be closed IMHO.

avatar brianteeman
brianteeman - comment - 18 Feb 2016

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

avatar brianteeman brianteeman - change - 18 Feb 2016
Status Confirmed Closed
Closed_Date 0000-00-00 00:00:00 2016-02-18 10:29:47
Closed_By brianteeman
avatar brianteeman brianteeman - close - 18 Feb 2016
avatar brianteeman brianteeman - close - 18 Feb 2016
avatar richard67
richard67 - comment - 18 Feb 2016

#9091 and #9092 meanwhile are merged too, so the things mentioned by this issue here are solved.


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

Add a Comment

Login with GitHub to post a comment