[ 
http://jira.magnolia-cms.com/browse/MGNLUI-407?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Daniel Lipp updated MGNLUI-407:
-------------------------------

        Summary: Adapt ActionbarPresenter to use varargs for methods normally 
called multiple times in a row  (was: Adapt ActionbarPresenter#enable(String) 
and disable(String) to use varargs.)
    Description: 
Currently ActionbarPresenter e.g. only offers enable(String) and 
disable(String). In most scenarios there's actually a few actions that have to 
be enabled or disabled. Having enable / disable with varargs String (String...) 
as param could make this code much nicer.

e.g.             
actionbar.enable("addFolder");
actionbar.enable("addNode");
actionbar.enable("addProperty");
actionbar.enable("delete");

could be written as 
actionbar.enable("addFolder", "addNode", "addProperty", "delete");



  was:
Currently ActionbarPresenter only offers enable(String) and disable(String). In 
most scenarios there's actually a few actions that have to be enabled or 
disabled. Having enable / disable with varargs String (String...) as param 
could make this code much nicer.

e.g.             
actionbar.enable("addFolder");
actionbar.enable("addNode");
actionbar.enable("addProperty");
actionbar.enable("delete");

could be written as 
actionbar.enable("addFolder", "addNode", "addProperty", "delete");




> Adapt ActionbarPresenter to use varargs for methods normally called multiple 
> times in a row
> -------------------------------------------------------------------------------------------
>
>                 Key: MGNLUI-407
>                 URL: http://jira.magnolia-cms.com/browse/MGNLUI-407
>             Project: Magnolia UI
>          Issue Type: Improvement
>      Security Level: Public
>    Affects Versions: 5.0 Alpha1
>            Reporter: Daniel Lipp
>             Fix For: 5.0
>
>
> Currently ActionbarPresenter e.g. only offers enable(String) and 
> disable(String). In most scenarios there's actually a few actions that have 
> to be enabled or disabled. Having enable / disable with varargs String 
> (String...) as param could make this code much nicer.
> e.g.             
> actionbar.enable("addFolder");
> actionbar.enable("addNode");
> actionbar.enable("addProperty");
> actionbar.enable("delete");
> could be written as 
> actionbar.enable("addFolder", "addNode", "addProperty", "delete");

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.magnolia-cms.com/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        


----------------------------------------------------------------
For list details, see: http://www.magnolia-cms.com/community/mailing-lists.html
Alternatively, use our forums: http://forum.magnolia-cms.com/
To unsubscribe, E-mail to: <[email protected]>
----------------------------------------------------------------

Reply via email to