? Success

User tests: Successful: Unsuccessful:

avatar JoomliC
JoomliC
17 Jan 2014

default_ordering_button

A button to save the current list ordering as default in admin.
This button sets this ordering for all users with access to admin, and is only visible to users with core.admin permission (access to options).
The settings are saved in params (hidden fields in config file)
Currently, only added in com_content, but could be added in other lists (users, categories, and every list of Joomla).
Advice, comments or help from contributors are welcome!

Request Issue: [#33108] Add an option to set list order by default - admin list
link: http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=33108&start=625

810c0dd 17 Dec 2013 avatar JoomliC 3.2.1
f89a397 9 Jan 2014 avatar JoomliC back
avatar JoomliC JoomliC - open - 17 Jan 2014
avatar Hackwar
Hackwar - comment - 3 Mar 2014

Sorry, but this PR as well as your other has several additional commits that are not part of what you are proposing. Please open a new, clean PR.

My personal opinion is, that I wouldn't want to have this feature. When weighing the benefit of this new feature and the cost by adding more parameters, I would vote to not include this.

avatar Bakual
Bakual - comment - 3 Mar 2014

As already said in the original PR, I'm not a big fan of the button to save the params. Imho it only makes code a lot more complicate and may confuse the user which likely expects the setting to be for himself only, when it's in fact for all users.
If we want to add new params they should be in the configuration page, and not as a button in the list itself.
Also, adding component specific code into a JLayout looks strange to me. It sort of defeats the purpose of JLayouts.

So please either update this PR with a solution which addresses those issues, or close this one and create a new one.

avatar JoomliC
JoomliC - comment - 3 Mar 2014
Sorry, but this PR as well as your other has several additional commits that are not part of what you are proposing. Please open a new, clean PR.

Hi!
My first Pr were processed with my git client, giving in fact too many problems. Since, my lastest PR are much more clean!
Really sorry for inconvenience, and thank you to forgive a "newbie" in contribution! ;)

My personal opinion is, that I wouldn't want to have this feature. When weighing the benefit of this new feature and the cost by adding more parameters, I would vote to not include this.
So please either update this PR with a solution which addresses those issues, or close this one and create a new one.

Thank you for your advice and opinions!
I closed this PR, and when a little time, will work on a better advanced solution to integrate this possibility inside core, or by a third party extension...
The opinion of seasoned contributor is most interesting!

Best Regards,
Cyril

avatar JoomliC JoomliC - close - 3 Mar 2014

Add a Comment

Login with GitHub to post a comment