Tapestry 5
  1. Tapestry 5
  2. TAP5-261

Palette option reordering broken in IE

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.0.16
    • Fix Version/s: 5.0.16
    • Component/s: None
    • Labels:
      None

      Description

      The re-ording buttons don't work in IE (6 or 7; probably not for 5, but untested). The issue is the same as for TAPESTRY-1745, but that issue affected transferring options between selects, and this issue is for reordering objects within selects. The root problem is the same, though, just in a different place in the .js.

      1. TAP5-261.patch
        1 kB
        Robert Zeigler

        Issue Links

          Activity

          Hide
          Howard M. Lewis Ship added a comment -

          Thanks for the patch. Tested under IE 7.

          Show
          Howard M. Lewis Ship added a comment - Thanks for the patch. Tested under IE 7.
          Hide
          Robert Zeigler added a comment -

          Patch that fixes the issue. Refactors the code a bit so both moveOption and reorderOption call into the same code for adding the option.

          Show
          Robert Zeigler added a comment - Patch that fixes the issue. Refactors the code a bit so both moveOption and reorderOption call into the same code for adding the option.

            People

            • Assignee:
              Howard M. Lewis Ship
              Reporter:
              Robert Zeigler
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development