? ? Success

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
19 Jul 2015

This PR (thanks to @rcheesley) simplifies the text for this field which is seen on the permissions tab for all components.

Before

before

After

after

Testing

The easiest way to check this is to go to Global configuration and then work through all the components on the left hand side and check their Permissions tab


Extra Reading on Eliminate redundancy and Avoiding over-communication
https://msdn.microsoft.com/en-gb/library/windows/desktop/dn742478.aspx
Eliminate redundancy
Redundant text not only takes valuable screen space, but weakens the effectiveness of the important ideas or actions that you are trying to convey. It is also a waste of the reader's time, and all the more so in a context where scanning is the norm. Windows strives to explain what users need to do once—well and concisely.
Review each window and eliminate duplicate words and statements, both within and across controls. Don't avoid important text—be explicit wherever necessary—but don't be redundant and don't explain the obvious.
Avoid over-communication
Even if text isn't redundant, it can simply be too wordy in an effort to explain every detail. Too much text discourages reading—the eye tends to skip right over it—ironically resulting in less communication rather than more. In UI text, concisely communicate the essential information. If more information is necessary for some users or some scenarios, provide a link to more detailed Help content, or perhaps to a glossary entry for clarification of a term.

avatar brianteeman brianteeman - open - 19 Jul 2015
avatar brianteeman brianteeman - change - 19 Jul 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 19 Jul 2015
Labels Added: ? ?
avatar MAT978 MAT978 - test_item - 19 Jul 2015 - Tested successfully
avatar brianteeman brianteeman - change - 19 Jul 2015
Category Language & Strings UI/UX
avatar brianteeman brianteeman - change - 19 Jul 2015
Easy No Yes
avatar MAT978
MAT978 - comment - 19 Jul 2015

#7469 works as described (wasn't able from patchtester).
Great improvement!
Thanks.


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

avatar spignataro spignataro - test_item - 19 Jul 2015 - Tested successfully
avatar jwaisner
jwaisner - comment - 19 Jul 2015

@test

PR works as intended. Always happy to reduce clutter.


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

avatar jwaisner jwaisner - test_item - 19 Jul 2015 - Tested successfully
avatar Bakual
Bakual - comment - 19 Jul 2015

It's not exactly correct.
Especially Inherit is either coming from global setting or from the parent group. If either allows the action, it is allowed.
For Denied, it ignores whatever is set in global setting and parent group. It's also worth to know that it will also deny the action for all children fo the group.

Other than that, it looks fine.

e7ca27f 20 Jul 2015 avatar brianteeman NOTE
avatar brianteeman
brianteeman - comment - 20 Jul 2015

rebased to resolve merge conflicts

avatar brianteeman
brianteeman - comment - 22 Jul 2015

@RCheesley do you want to take another pass at the text in light of @bakual comments (when you are back from holiday)


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

avatar zero-24 zero-24 - alter_testresult - 27 Jul 2015 - MAT978: Not tested
avatar zero-24 zero-24 - alter_testresult - 27 Jul 2015 - jwaisner: Not tested
avatar zero-24 zero-24 - alter_testresult - 27 Jul 2015 - spignataro: Not tested
avatar brianteeman
brianteeman - comment - 4 Aug 2015

Suggested change from @RCheesley taking into account @bakual comments

  1. Changes apply to this component only.
    Inherited - a Global Configuration setting or higher level setting is applied.
    Denied always wins - whatever is set at the Global or higher level and applies to all child elements.
    Allowed will enable the action for this component unless it is overruled by a Global Configuration setting.

???


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

avatar Bakual
Bakual - comment - 4 Aug 2015

Yep, sounds accurate this way.

avatar brianteeman
brianteeman - comment - 4 Aug 2015

OK I will update the PR accordingly

avatar brianteeman
brianteeman - comment - 5 Aug 2015

PR updated as above

avatar Bakual
Bakual - comment - 5 Aug 2015

Looks fine, thanks!

avatar Bakual Bakual - reference | 3938270 - 5 Aug 15
avatar Bakual Bakual - merge - 5 Aug 2015
avatar Bakual Bakual - close - 5 Aug 2015
avatar brianteeman brianteeman - head_ref_deleted - 5 Aug 2015
avatar mbabker mbabker - reference | f4d413b - 5 Aug 15

Add a Comment

Login with GitHub to post a comment