ICEfaces
  1. ICEfaces
  2. ICE-6001

JavaScript error when opening a popup that has an inputRichText component in IE

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.8.2-EE-GA_P01
          • Fix Version/s: 1.8.3, 1.8.2-EE-GA_P02
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            IE
          • Workaround Exists:
            Yes
          • Workaround Description:
            Hide
            Workaround is to avoid form-level updates by wrapping the panelPopup inside a panelGroup:
             
             <ice:panelGroup >
                            <ice:panelPopup >
            </ice:panelGroup>
            Show
            Workaround is to avoid form-level updates by wrapping the panelPopup inside a panelGroup:    <ice:panelGroup >                 <ice:panelPopup > </ice:panelGroup>

            Description

            When an inputRichText component is included in a panelPopup a JavaScript error is thrown when the popup is opened. The error is 'submit.tagName' is null or not an object. This error isn't thrown when the popup uses the visible attribute and not the rendered attribute to show/hide it self.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              yip.ng added a comment - - edited

              Can't reproduce, not in IE8, not in IE7. See video at http://screencast.com/t/c5ykLcdI.

              See screenshot 3 for the markup used.

              Show
              yip.ng added a comment - - edited Can't reproduce, not in IE8, not in IE7. See video at http://screencast.com/t/c5ykLcdI . See screenshot 3 for the markup used.
              Hide
              Permalink
              Arran Mccullough added a comment -

              Attached test case built for Tomcat 6 and source code.

              Show
              Arran Mccullough added a comment - Attached test case built for Tomcat 6 and source code.
              Hide
              Permalink
              yip.ng added a comment - - edited

              The error is inside the fckeditor code (all scrambled). See screenshot 4. Examination of the stack trace shows no calls from our JS. Must be some init code from within fckeditor itself. Without the unscrambled source it's hard to debug.

              Show
              yip.ng added a comment - - edited The error is inside the fckeditor code (all scrambled). See screenshot 4. Examination of the stack trace shows no calls from our JS. Must be some init code from within fckeditor itself. Without the unscrambled source it's hard to debug.
              Hide
              Permalink
              Ken Fyten added a comment -

              Adnan is upgrading our FCKeditor version, I'll give this to him to examine once he's finished that.

              Show
              Ken Fyten added a comment - Adnan is upgrading our FCKeditor version, I'll give this to him to examine once he's finished that.
              Hide
              Permalink
              Adnan Durrani added a comment -

              The error is inside one of a FCKEditor file, which can not be fixed. It came across that is due to the reason that when form level update comes it causes this error only in IE, seems like FCKEditor doesn't like the way form element gets inserted.

              However there is a workaround, wrapping panelPopup inside a pbelGroup fixes this issue.
              <ice:panelGroup>
              <ice:panelPopup .>

              Show
              Adnan Durrani added a comment - The error is inside one of a FCKEditor file, which can not be fixed. It came across that is due to the reason that when form level update comes it causes this error only in IE, seems like FCKEditor doesn't like the way form element gets inserted. However there is a workaround, wrapping panelPopup inside a pbelGroup fixes this issue. <ice:panelGroup> <ice:panelPopup .>

                People

                • Assignee:
                  Adnan Durrani
                  Reporter:
                  Arran Mccullough
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: