Uploaded image for project: 'Magnolia UI'
  1. Magnolia UI
  2. MGNLUI-407

Adapt ActionbarPresenter to use varargs as params for methods often called multiple times in a row

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Fixed
    • Neutral
    • 5.0
    • 5.0
    • None
    • None

    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");

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            Activity

              People

                dlipp Daniel Lipp
                dlipp Daniel Lipp
                Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  Checklists

                    Task DoD