ICEfaces
  1. ICEfaces
  2. ICE-7895

Update jQuery to 1.7.1, jQuery UI 1.8.18 and Wijmo 2.0.5

    Details

    • Type: Task Task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.1.0.BETA1, 3.1
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      *
    • Assignee Priority:
      P2
    • Affects:
      Compatibility/Configuration

      Description

      Update to jQuery 1.7.1, jQuery UI 1.8.18 and Wijmo 2.0.5 to fix bugs like ICE-7665 and for other general improvements.

        Issue Links

          Activity

          Hide
          Arturo Zambrano added a comment -

          Committed update to jQuery 1.7.2 at revision 28901. Also modified occurrences of calls to live() for on(), and calls to die() for off().

          The following issues were found after this update:

          • The list component is slower.
          • Panel and row expansion feel slower as well.
          • It's not possible to do multi-column sorting (even before removing calls to live()).
          • There's an issue with multiple cell selection when ajax is used (it highlights the cell multiple times when many of them have been selected).
          Show
          Arturo Zambrano added a comment - Committed update to jQuery 1.7.2 at revision 28901. Also modified occurrences of calls to live() for on(), and calls to die() for off(). The following issues were found after this update: The list component is slower. Panel and row expansion feel slower as well. It's not possible to do multi-column sorting (even before removing calls to live()). There's an issue with multiple cell selection when ajax is used (it highlights the cell multiple times when many of them have been selected).
          Hide
          Arturo Zambrano added a comment -

          Committed final update, consisting of jQuery 1.7.1, jQuery UI 1.8.18 and wijmo 2.0.5, at revision 28922.

          Had to update to jQuery 1.7.1 (instead of 1.7.2) since that's the version supported by the latest version of wijmo. Also, jQuery UI 1.8.18 works better than the 1.8.20 version.

          The issue with ace:contextMenu is now fixed with this commit. However, there's still a minor issue in the ICEfaces showcase: in the table integration demo, after one activates a menu item in the context menu (either add rows or undo), the contextMenu doesn't appear again.

          These issues are also present at the moment:

          • In the ace:confirmationDialog showcase demos, every time the button is clicked to show a dialog, all the dialogs that have been opened at some point in the session appear at the same time.
          • There's a Javascript error thrown when trying to activate the ace:resizable listener.
          • ace:dialog and ace:confirmationDialog have larger fonts.
          Show
          Arturo Zambrano added a comment - Committed final update, consisting of jQuery 1.7.1, jQuery UI 1.8.18 and wijmo 2.0.5, at revision 28922. Had to update to jQuery 1.7.1 (instead of 1.7.2) since that's the version supported by the latest version of wijmo. Also, jQuery UI 1.8.18 works better than the 1.8.20 version. The issue with ace:contextMenu is now fixed with this commit. However, there's still a minor issue in the ICEfaces showcase: in the table integration demo, after one activates a menu item in the context menu (either add rows or undo), the contextMenu doesn't appear again. These issues are also present at the moment: In the ace:confirmationDialog showcase demos, every time the button is clicked to show a dialog, all the dialogs that have been opened at some point in the session appear at the same time. There's a Javascript error thrown when trying to activate the ace:resizable listener. ace:dialog and ace:confirmationDialog have larger fonts.
          Hide
          Arturo Zambrano added a comment -

          The issue with ace:resizable is not related to this jQuery update. The fix is straight forward, and it is now documented in ICE-8070.

          Show
          Arturo Zambrano added a comment - The issue with ace:resizable is not related to this jQuery update. The fix is straight forward, and it is now documented in ICE-8070 .
          Hide
          Arturo Zambrano added a comment -

          Also replaced calls to live() for on() to improve performance.

          Show
          Arturo Zambrano added a comment - Also replaced calls to live() for on() to improve performance.

            People

            • Assignee:
              Arturo Zambrano
              Reporter:
              yip.ng
            • Votes:
              1 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: