? Success

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
12 Jan 2015

This PR sorts the template positions in the module manager when selecting the position for the template by template and then by position. It also takes into account the translated name of the position.

How to test

  1. Go into the module manager, edit a module, see that the positions are not in alphabetical order.
  2. Apply patch
  3. See that the positions are ordered (and grouped) by template first and then in the respective template group, ordered alphabetically by position.
avatar Hackwar Hackwar - open - 12 Jan 2015
avatar jissues-bot jissues-bot - change - 12 Jan 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 12 Jan 2015
Category Administration UI/UX
avatar zero-24 zero-24 - change - 12 Jan 2015
Easy No Yes
avatar brianteeman
brianteeman - comment - 12 Jan 2015

Aren't they in the order that they are listed in the templateDetails.xml
file

A template creator may have placed them in specific order and this will
change that for example I might have grouped the positions in an order that
they are displayed moving down the page

  • Header
  • Menu
  • Top
  • Left
  • Right
  • Footer

On 12 January 2015 at 12:22, Hannes Papenberg notifications@github.com
wrote:

This PR sorts the template positions in the module manager when selecting
the position for the template by template and then by position. It also
takes into account the translated name of the position.
How to test

  1. Go into the module manager, edit a module, see that the positions are not in alphabetical order.
  2. Apply patch
  3. See that the positions are ordered (and grouped) by template first and then in the respective template group, ordered alphabetically by position.

You can merge this Pull Request by running

git pull https://github.com/Hackwar/joomla-cms modulepositionordering

Or view, comment on, or merge it at:

#5682
Commit Summary

  • Sort Template positions in drop down

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#5682.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar Hackwar
Hackwar - comment - 12 Jan 2015

Yes, they are displayed in the order that they are in the XML. And yes, it might be in the order that they appear in the template. I would however argue that it is a lot simpler for the user when the positions are in alphabetical order. Especially for a user that already knows which position he wants, which I would argue is the case in 90% of the cases, it's quicker to look out for the alphabetical order than the other way around. When I know I want "Right", its quicker for me to find the position under R, than to remember that it is below "heading" and "left", but above "content-top".

That would also assume that all template developers consciously order/group the positions in the XML.

avatar brianteeman
brianteeman - comment - 12 Jan 2015

I am not saying its wrong to change to alphasort. Personally I add
positions to my xml as i create them so they are ordered - maybe others
don't. Maybe it is better to have them alphasorted. I am just pointing out
that there is a legitimate use case for them not being alphasorted

On 12 January 2015 at 12:56, Hannes Papenberg notifications@github.com
wrote:

Yes, they are displayed in the order that they are in the XML. And yes, it
might be in the order that they appear in the template. I would however
argue that it is a lot simpler for the user when the positions are in
alphabetical order. Especially for a user that already knows which position
he wants, which I would argue is the case in 90% of the cases, it's quicker
to look out for the alphabetical order than the other way around. When I
know I want "Right", its quicker for me to find the position under R, than
to remember that it is below "heading" and "left", but above "content-top".

That would also assume that all template developers consciously
order/group the positions in the XML.


Reply to this email directly or view it on GitHub
#5682 (comment).

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar richard67
richard67 - comment - 12 Jan 2015

@test Success.
screen shot 2015-01-12 at 14 30 48
... And no PHP warning, too ;-)


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

avatar richard67 richard67 - test_item - 12 Jan 2015 - Tested successfully
avatar waader
waader - comment - 14 Jan 2015

@test works!

avatar waader waader - test_item - 14 Jan 2015 - Tested successfully
avatar zero-24 zero-24 - change - 14 Jan 2015
Status Pending Ready to Commit
avatar zero-24
zero-24 - comment - 14 Jan 2015

Moving to RTC Thanks to all!


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

avatar brianteeman brianteeman - change - 14 Jan 2015
Labels Added: ?
avatar rdeutz rdeutz - close - 15 Jan 2015
avatar zero-24 zero-24 - close - 15 Jan 2015
avatar rdeutz rdeutz - change - 15 Jan 2015
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2015-01-15 21:47:19
avatar zero-24 zero-24 - change - 14 Oct 2015
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment