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

Allow multiple reorder actions with keyboard on MultiValueField

    XMLWordPrintable

Details

    • Yes
    • Saigon 61
    • 3

    Description

      We just noticed you added re-ordering to the MultiValueFieldDefinition. It's great to have! We had already implemented it on our own, so we are going to eliminate our subclass and use yours. The only thing we did that you didn't was the following:

      When you use keyboard navigation to tab over to one of the move up or move down links, and activate the link, it loses focus. So if I want to move something up three spaces, now I need to tab all the way down to the link two more times. Instead we should just maintain focus on the same link so you can activate the link 3 times and you're done. Hopefully that's a clear explanation.

      I'm including a really simple patch against today's trunk to illustrate. Untested but it's dead simple and really close to what we did in our implementation.

      Checklists

        Acceptance criteria

        Attachments

          Activity

            People

              hieu.nguyen Hieu Nguyen Duc
              nwing Nickolaus Wing
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Checklists

                  Task DoD

                  Time Tracking

                    Estimated:
                    Original Estimate - 2d
                    2d
                    Remaining:
                    Remaining Estimate - 0d
                    0d
                    Logged:
                    Time Spent - 1d 1.5h Time Not Required
                    1d 1.5h