Uploaded image for project: 'Camunda Optimize'
  1. Camunda Optimize
  2. OPT-1421

update multiple selection component to make it usable combined selection

XMLWordPrintable

    • Icon: Task Task
    • Resolution: Fixed
    • Icon: L3 - Default L3 - Default
    • 2.2.0
    • None
    • None
    • None

      Currently, Combined Report Selection Panel uses its own selection panel instead of using "TypeAheadMultipleSelection" component since there are a couple of differences between them.

      The goal is to make "TypeAheadMultipleSelection" also usable for combined selection Panel instead of having the same functionality repeated twice.

        This is the controller panel for Smart Panels app

            [OPT-1421] update multiple selection component to make it usable combined selection

            Sebastian Stamm added a comment - - edited
            • The formatter for StringInput is unnecessary as it is just the default formatter
            • The value of the checkbox DOM element is now "[object Object]"
            • I would like to avoid creating new event listeners on every render of the TypeaheadMultipleSelection. My proposal would be to set the value of the checkbox to the index of the entry in the array and use an internal event handler parsing the value and state from the event, resolving the index to the actual element and passing the element and selection state to the parent handler.
            • The placeholder for the text input field should use the label instead of being "Search for an entry" it should be "Search for {label}"

            Sebastian Stamm added a comment - - edited The formatter for StringInput is unnecessary as it is just the default formatter The value of the checkbox DOM element is now "[object Object]" I would like to avoid creating new event listeners on every render of the TypeaheadMultipleSelection. My proposal would be to set the value of the checkbox to the index of the entry in the array and use an internal event handler parsing the value and state from the event, resolving the index to the actual element and passing the element and selection state to the parent handler. The placeholder for the text input field should use the label instead of being "Search for an entry" it should be "Search for {label}"

              Unassigned Unassigned
              omran.abazeed Omran Abazeed
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: