Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-38889

User configurable option for number of options displayed in Active Choices

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Open (View Workflow)
    • Priority: Minor
    • Resolution: Unresolved
    • Component/s: active-choices-plugin
    • Labels:
      None
    • Environment:
      Active Choices plugin v 1.4
    • Similar Issues:

      Description

      Sometimes it is better that an AC control displays more of the available options even if a single selection will be made. For example, the AC display of files in a folder benefits from displaying more than a single file even though a single file will be selected by the user. Currently, an expanded display (displaying 5 option values) is available only for an AC multi-select control. Perhaps, we can introduce a user-selectable configuration option that controls the number of the displayed options.

        Attachments

          Activity

          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Bruno P. Kinoshita
          Path:
          src/main/java/org/biouno/unochoice/AbstractCascadableParameter.java
          src/main/java/org/biouno/unochoice/AbstractScriptableParameter.java
          src/main/java/org/biouno/unochoice/CascadeChoiceParameter.java
          src/main/java/org/biouno/unochoice/ChoiceParameter.java
          src/main/java/org/biouno/unochoice/DynamicReferenceParameter.java
          src/main/resources/org/biouno/unochoice/CascadeChoiceParameter/config.jelly
          src/main/resources/org/biouno/unochoice/ChoiceParameter/config.jelly
          src/main/resources/org/biouno/unochoice/DynamicReferenceParameter/config.jelly
          src/main/resources/org/biouno/unochoice/common/checkboxContent.jelly
          src/main/resources/org/biouno/unochoice/common/choiceParameterCommon.jelly
          src/main/resources/org/biouno/unochoice/common/radioContent.jelly
          src/main/webapp/help-useDefaultMaxVisibleItemCount.html
          src/main/webapp/help-visibleItemCount.html
          src/test/java/org/biouno/unochoice/issue38889/TestNumberOfVisibleElements.java
          src/test/java/org/biouno/unochoice/issue38889/package-info.java
          http://jenkins-ci.org/commit/active-choices-plugin/83f53bd14f939711e56213ca91ad679fffedd7e6
          Log:
          JENKINS-38889 Add an option to disable the default maximum number of visible elements. Also add another option to specify the number of visible elements on the UI. Unit tests included.

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Bruno P. Kinoshita Path: src/main/java/org/biouno/unochoice/AbstractCascadableParameter.java src/main/java/org/biouno/unochoice/AbstractScriptableParameter.java src/main/java/org/biouno/unochoice/CascadeChoiceParameter.java src/main/java/org/biouno/unochoice/ChoiceParameter.java src/main/java/org/biouno/unochoice/DynamicReferenceParameter.java src/main/resources/org/biouno/unochoice/CascadeChoiceParameter/config.jelly src/main/resources/org/biouno/unochoice/ChoiceParameter/config.jelly src/main/resources/org/biouno/unochoice/DynamicReferenceParameter/config.jelly src/main/resources/org/biouno/unochoice/common/checkboxContent.jelly src/main/resources/org/biouno/unochoice/common/choiceParameterCommon.jelly src/main/resources/org/biouno/unochoice/common/radioContent.jelly src/main/webapp/help-useDefaultMaxVisibleItemCount.html src/main/webapp/help-visibleItemCount.html src/test/java/org/biouno/unochoice/issue38889/TestNumberOfVisibleElements.java src/test/java/org/biouno/unochoice/issue38889/package-info.java http://jenkins-ci.org/commit/active-choices-plugin/83f53bd14f939711e56213ca91ad679fffedd7e6 Log: JENKINS-38889 Add an option to disable the default maximum number of visible elements. Also add another option to specify the number of visible elements on the UI. Unit tests included.
          Hide
          kinow Bruno P. Kinoshita added a comment -

          Good points.

          Added two options, under each parameter type. The options are in the Advanced section.

          The checkbox should be checked by default. It is a flag that when unchecked disables a verification step that checks whether the number of elements in the return array is bigger than 10.

          The input text is used for the number of visible items displayed. If the checkbox is enabled, it has no effect. As the default behavior doesn't allow to change the number of visible elements.

          But when the checkbox is unchecked, this value is used to specify the number of visible elements.

          Code in a branch in GitHub. Attached binary plug-in file. Could you try it locally, see if that makes sense, and if that works as expected, please Ioannis Moutsatsos

          Show
          kinow Bruno P. Kinoshita added a comment - Good points. Added two options, under each parameter type. The options are in the Advanced section. The checkbox should be checked by default. It is a flag that when unchecked disables a verification step that checks whether the number of elements in the return array is bigger than 10. The input text is used for the number of visible items displayed. If the checkbox is enabled, it has no effect. As the default behavior doesn't allow to change the number of visible elements. But when the checkbox is unchecked, this value is used to specify the number of visible elements. Code in a branch in GitHub. Attached binary plug-in file. Could you try it locally, see if that makes sense, and if that works as expected, please Ioannis Moutsatsos

            People

            • Assignee:
              kinow Bruno P. Kinoshita
              Reporter:
              ioannis Ioannis Moutsatsos
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated: