ICEfaces
  1. ICEfaces
  2. ICE-4292

Error switching between toolbar modes in Rich Input demo in Component Showcase with IE

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.8RC2
          • Fix Version/s: 1.8.1
          • Component/s: Sample Apps
          • Labels:
            None
          • Environment:
            IE7, IE 8 (compatible + non-compatible mode), (IE6 not tested)

            Description

            When viewing the inputRichText demo in IE7 or IE8, entering some text in the textedit box, saving it, and then switching to Basic toolbar mode (and then same thing back to Default toolbar mode) causes a JS error to popup (if JS errors are set to be displayed in Internet Option Advanced settings).

            If JS errors are not configured to be displayed, no error dialog appears, but after this happens you cannot set focus to the edit textarea of the component anymore in Default mode. Actually, testing shows that once the error happens you can't focus on ANY input text components at all until you do a reload. Even those in another demo such as Text Entry demo.

              Issue Links

              blocks

              Bug - A problem which impairs or prevents the functions of the product. ICE-4366 JavaScript error in IE when inputRichtext is in a panelPopup

              • Major - Major loss of function.
              • Closed

              Bug - A problem which impairs or prevents the functions of the product. ICE-5215 JavaScript error in IE when inputtext onchange is in a panelPopup

              • Major - Major loss of function.
              • Closed
              duplicates

              Bug - A problem which impairs or prevents the functions of the product. ICE-4366 JavaScript error in IE when inputRichtext is in a panelPopup

              • Major - Major loss of function.
              • Closed

              Bug - A problem which impairs or prevents the functions of the product. ICE-5215 JavaScript error in IE when inputtext onchange is in a panelPopup

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                Sam Xiao added a comment -

                To reproduce the issue, you need to input something in the edit area under default mode and then switch to basic mode

                Show
                Sam Xiao added a comment - To reproduce the issue, you need to input something in the edit area under default mode and then switch to basic mode
                Hide
                Permalink
                Adnan Durrani added a comment -

                To update the contents of the inputRichText component, we use the following FCKEditor's API:
                editorInstance.SetHTML(value);

                Seems like the call of the above method causing this javascript error under IE8.

                The debugger shows that it is due to some security issue with Iframe contents. Please see the attached image. IE8-DEBUG.jpg.

                However if I used the pure HTML based demo using FCKEditor, and call the same function to update the contents. It works fine, and no error.

                Show
                Adnan Durrani added a comment - To update the contents of the inputRichText component, we use the following FCKEditor's API: editorInstance.SetHTML(value); Seems like the call of the above method causing this javascript error under IE8. The debugger shows that it is due to some security issue with Iframe contents. Please see the attached image. IE8-DEBUG.jpg. However if I used the pure HTML based demo using FCKEditor, and call the same function to update the contents. It works fine, and no error.
                Hide
                Permalink
                Adnan Durrani added a comment -

                Fixed :

                Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\inputrichtext\InputRichTextRenderer.java
                Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\inputrichtext\fckeditor_ext.js
                Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\inputrichtext\InputRichTextRenderer.java
                Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\inputrichtext\fckeditor_ext.js
                Completed: At revision: 18810

                Show
                Adnan Durrani added a comment - Fixed : Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\inputrichtext\InputRichTextRenderer.java Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\inputrichtext\fckeditor_ext.js Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\inputrichtext\InputRichTextRenderer.java Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\inputrichtext\fckeditor_ext.js Completed: At revision: 18810
                Hide
                Permalink
                Joanne Bai added a comment -

                Verified with success on ICEfaces 1.8.1 build 1 + Glassfish v2.1 using component showcase facelets-enh

                Browsers tested: IE6, IE7, IE8

                Show
                Joanne Bai added a comment - Verified with success on ICEfaces 1.8.1 build 1 + Glassfish v2.1 using component showcase facelets-enh Browsers tested: IE6, IE7, IE8

                  People

                  • Assignee:
                    Arturo Zambrano
                    Reporter:
                    Sam Xiao
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    0 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: