ICEfaces-EE
  1. ICEfaces-EE
  2. IPCK-386

Some EE Accessible Components throw JavaScript errors when used with JavaScript enabled

    Details

    • Assignee Priority:
      P2

      Description

      The samples all look good with the exception of the ee-accessible-showcase. I tested in IE9 / Firefox11 / Chrome17.

      inputText:
      Typing in the fields or tabbing through them causes a JS error in IE9 and Firefox:
      IE9:
      Message: Exception thrown and not caught
      Line: 1
      Char: 16245
      Code: 0
      URI: http://localhost:8080/ee-accessible-showcase/javax.faces.resource/compat.js.jsf

      Firefox:
      Error: uncaught exception: Cannot find enclosing form.

      panelCollapsible:
      Clicking on the panel headers to collapse or expand them causes a JS error in IE9:
      Message: Invalid procedure call or argument
      Line: 1
      Char: 1
      Code: 0
      URI: http://localhost:8080/ee-accessible-showcase/component/panelCollapsible.jsf

      Some of the links in the panel are broken (ICEfaces SVN, ICEfaces Projects).

      panelTab:
      Switching tabs causes a JS error in IE9:
      Message: Invalid procedure call or argument
      Line: 1
      Char: 1
      Code: 0
      URI: http://localhost:8080/ee-accessible-showcase/component/panelTab.jsf

      Besides this there are no errors.

        Activity

        Ken Fyten created issue -
        Ken Fyten made changes -
        Field Original Value New Value
        Salesforce Case []
        Assignee Priority P1
        Assignee Ken Fyten [ ken.fyten ] yip.ng [ yip.ng ]
        Hide
        Ken Fyten added a comment -

        Resolved issues with panelCollapsible and panelTab by removing extraneous JavaScript in the renderers.

        There doesn't seem to be any inputText source in the accessible package, so will try removing that tag altogether.

        Show
        Ken Fyten added a comment - Resolved issues with panelCollapsible and panelTab by removing extraneous JavaScript in the renderers. There doesn't seem to be any inputText source in the accessible package, so will try removing that tag altogether.
        Hide
        yip.ng added a comment - - edited

        input text, input text area and input secret removed.

        For why they had been kept, see last point of http://jira.icesoft.org/browse/IPCK-351?focusedCommentId=38625&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#action_38625 and point 3 of http://jira.icesoft.org/browse/IPCK-351?focusedCommentId=38668&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#action_38668

        Modified: C:\svn\repo\icefaces-ee3\trunk\icefaces-ee\ee-comps\ee-accessible\component-metadata\src\main\resources\conf\extended-faces-config.xml
        Modified: C:\svn\repo\icefaces-ee3\trunk\icefaces-ee\ee-comps\ee-accessible\component-metadata\src\main\resources\conf\faces-config-base.xml
        Completed: At revision: 30946

        Show
        yip.ng added a comment - - edited input text, input text area and input secret removed. For why they had been kept, see last point of http://jira.icesoft.org/browse/IPCK-351?focusedCommentId=38625&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#action_38625 and point 3 of http://jira.icesoft.org/browse/IPCK-351?focusedCommentId=38668&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#action_38668 Modified: C:\svn\repo\icefaces-ee3\trunk\icefaces-ee\ee-comps\ee-accessible\component-metadata\src\main\resources\conf\extended-faces-config.xml Modified: C:\svn\repo\icefaces-ee3\trunk\icefaces-ee\ee-comps\ee-accessible\component-metadata\src\main\resources\conf\faces-config-base.xml Completed: At revision: 30946
        Ken Fyten made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Cruz Miraback added a comment -

        The same JS errors for panelCollapsible and panelTab are still occurring in IE9 using EE3.0 Build 10.

        Show
        Cruz Miraback added a comment - The same JS errors for panelCollapsible and panelTab are still occurring in IE9 using EE3.0 Build 10.
        Cruz Miraback made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s EE-3.1.0.GA [ 10326 ]
        Fix Version/s EE-3.0.0.GA [ 10261 ]
        yip.ng made changes -
        Attachment screenshot-1.png [ 14250 ]
        Hide
        yip.ng added a comment - - edited

        The collapsible or tab header is just a simple submit-type input element. There is already no onclick attribute attached to it. But the bridge seems to have registered a capture submit listener, and the error is from the bridge listener. See screenshot-1.png.

        Show
        yip.ng added a comment - - edited The collapsible or tab header is just a simple submit-type input element. There is already no onclick attribute attached to it. But the bridge seems to have registered a capture submit listener, and the error is from the bridge listener. See screenshot-1.png.
        Ken Fyten made changes -
        Salesforce Case []
        Assignee Priority P1 P2
        Assignee yip.ng [ yip.ng ] Mircea Toma [ mircea.toma ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #28587 Thu Mar 29 11:01:10 MDT 2012 mircea.toma IPCK-386 Add second check for IE9 which will verify if the looked up element has the 'id' attribute.
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/core/src/main/javascript/application.js
        Hide
        Mircea Toma added a comment -

        When onsubmit event is captured our bridge tries to verify if the submitting element still exists after the update. If the element misses the 'id' attribute it will try to lookup the element by it's name as a fallback mechanism (since for input elements the name and id will match) and if nothing's returned the id is set to match the name attribute. In IE9 the browser will return the element even when the its ID attribute is missing. This confuses the fallback mechanism which will fail when further on tries to lookup the element by its ID.

        Show
        Mircea Toma added a comment - When onsubmit event is captured our bridge tries to verify if the submitting element still exists after the update. If the element misses the 'id' attribute it will try to lookup the element by it's name as a fallback mechanism (since for input elements the name and id will match) and if nothing's returned the id is set to match the name attribute. In IE9 the browser will return the element even when the its ID attribute is missing. This confuses the fallback mechanism which will fail when further on tries to lookup the element by its ID.
        Hide
        Mircea Toma added a comment -

        Add second check for IE9 which will verify if the looked up element has the 'id' attribute.

        Show
        Mircea Toma added a comment - Add second check for IE9 which will verify if the looked up element has the 'id' attribute.
        Mircea Toma made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Security Private [ 10001 ]
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s EE-3.0.0.GA_P01 [ 10328 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #28596 Fri Mar 30 08:34:58 MDT 2012 mircea.toma IPCK-386 Add second check for IE9 which will verify if the looked up element has the 'id' attribute.
        Files Changed
        Commit graph MODIFY /icefaces3/branches/icefaces-3.0.x-maintenance/icefaces/core/src/main/javascript/application.js
        Hide
        Mircea Toma added a comment -

        Applied fix to maintenance branch.

        Show
        Mircea Toma added a comment - Applied fix to maintenance branch.
        Mircea Toma made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Fix Version/s EE-3.2.0.BETA [ 10571 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Ken Fyten
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: