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
              Arran Mccullough created issue -
              Arran Mccullough made changes -
              Field Original Value New Value
              Salesforce Case [5007000000D4xVb]
              Arran Mccullough made changes -
              Attachment IE7-richText-JavaScript-Error1.JPG [ 12506 ]
              Attachment IE7-richText-JavaScript-Error2.JPG [ 12507 ]
              Ken Fyten made changes -
              Fix Version/s 1.8.2-EE-GA_P02 [ 10226 ]
              Fix Version/s 1.8.3 [ 10211 ]
              Ken Fyten made changes -
              Assignee Priority P2
              Assignee Yip Ng [ yip.ng ]
              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.
              yip.ng made changes -
              Attachment screenshot-03.png [ 12644 ]
              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.
              Arran Mccullough made changes -
              Attachment Case9373Example.war [ 12645 ]
              Attachment main.jspx [ 12646 ]
              Attachment TestBean.java [ 12647 ]
              yip.ng made changes -
              Attachment screenshot-04.png [ 12649 ]
              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.
              Ken Fyten made changes -
              Assignee Priority P2 P3
              Assignee Yip Ng [ yip.ng ] Adnan Durrani [ adnan.durrani ]
              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 .>
              Adnan Durrani made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Workaround Description Use the visible property to show/hide the popup. Workaround is to avoid form-level updates by wrapping the panelPopup inside a panelGroup:
               
               <ice:panelGroup >
                              <ice:panelPopup >
              </ice:panelGroup>
              Assignee Priority P3
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: