ICEfaces
  1. ICEfaces
  2. ICE-7078

Double Clicking on an empty cell in a table with a rowClickListener in IE9 causes a Javascript exception

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-1.8.2.GA_P03, 3.0
    • Fix Version/s: 3.0.1, EE-3.0.0.GA, EE-1.8.2.GA_P04
    • Component/s: Bridge
    • Labels:
      None
    • Environment:
      Linux 2.6.18-128.el5

      Description

      We have a dataTable with a rowSelector. It is coded, so that when the user double-clicks a row on the table, it navigates to the next page.

      If the user double clicks in a cell without a value, a javascript error is thrown: "Unknown runtime error". This only occurs in IE9. This error is not encountered when Firefox is used.

        Activity

        Hide
        Ed Hillmann added a comment -

        This was thrown by lib/extras.js, here

        Ice.registerDblClick = function(elem) {
        if (document.selection) document.selection.empty();
        if (Ice.clickEvents[elem.id])

        { Ice.clickEvents[elem.id].submit(2); }

        }

        It's the call to document.selection.empty(). I don't know what the problem is. This is a patch that simply catches the exception an ignores it. I don't know whether this is a long-term solution, but it allows my application to double-click on the row in the table, which is the primary goal for me.

        Show
        Ed Hillmann added a comment - This was thrown by lib/extras.js, here Ice.registerDblClick = function(elem) { if (document.selection) document.selection.empty(); if (Ice.clickEvents [elem.id] ) { Ice.clickEvents[elem.id].submit(2); } } It's the call to document.selection.empty(). I don't know what the problem is. This is a patch that simply catches the exception an ignores it. I don't know whether this is a long-term solution, but it allows my application to double-click on the row in the table, which is the primary goal for me.
        Hide
        Mircea Toma added a comment -

        The provided patch was applied to ignore "Unknown runtime error" in IE when there's nothing selected.

        Show
        Mircea Toma added a comment - The provided patch was applied to ignore "Unknown runtime error" in IE when there's nothing selected.

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Ed Hillmann
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: