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

          Activity

          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Tyler Lecouffe made changes -
          Link This issue blocks ICE-5215 [ ICE-5215 ]
          Tyler Lecouffe made changes -
          Link This issue duplicates ICE-5215 [ ICE-5215 ]
          Hide
          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
          Adnan Durrani made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Adnan Durrani made changes -
          Link This issue blocks ICE-4366 [ ICE-4366 ]
          Hide
          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
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #18810 Tue Apr 28 12:46:36 MDT 2009 adnan.durrani Fix for ICE-4292 (Error switching between toolbar modes in Rich Input demo in Component Showcase with IE )
          It fixes the ICE-4366 as well.
          Files Changed
          Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/inputrichtext/InputRichTextRenderer.java
          Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/inputrichtext/fckeditor_ext.js
          Ken Fyten made changes -
          Link This issue duplicates ICE-4366 [ ICE-4366 ]
          Ken Fyten made changes -
          Salesforce Case []
          Assignee Priority P2
          Assignee Adnan Durrani [ adnan.durrani ] Arturo Zambrano [ artzambrano ]
          Hide
          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.
          Adnan Durrani made changes -
          Attachment IE8-DEBUG.jpg [ 11652 ]
          Ken Fyten made changes -
          Link This issue blocks ICE-4290 [ ICE-4290 ]
          Ken Fyten made changes -
          Salesforce Case []
          Assignee Priority P3 P2
          Description When viewing the inputRichText demo in IE7 or IE8, entering a value in the edit box, without 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.
          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.
          Ken Fyten made changes -
          Salesforce Case []
          Security Private [ 10001 ]
          Ken Fyten made changes -
          Salesforce Case []
          Component/s Sample Apps [ 10010 ]
          Affects [Sample App./Tutorial]
          Ken Fyten made changes -
          Summary Rich input demo issue in IE 8 final Error switching between toolbar modes in Rich Input demo in Component Showcase with IE
          Environment IE 8 (compatible + non-compatible mode) IE7, IE 8 (compatible + non-compatible mode), (IE6 not tested)
          Salesforce Case []
          Fix Version/s 1.8.1 [ 10170 ]
          Affects [Sample App./Tutorial]
          Assignee Priority P2 P3
          Description When switching to basic mode, an error message box popped up in IE 8. When viewing the inputRichText demo in IE7 or IE8, entering a value in the edit box, without 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.
          Ken Fyten made changes -
          Link This issue blocks ICE-4290 [ ICE-4290 ]
          Ken Fyten made changes -
          Salesforce Case []
          Assignee Priority P2
          Security Private [ 10001 ]
          Assignee Adnan Durrani [ adnan.durrani ]
          Sam Xiao made changes -
          Environment IE 8 IE 8 (compatible + non-compatible mode)
          Salesforce Case []
          Hide
          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
          Sam Xiao made changes -
          Field Original Value New Value
          Attachment IE8-richinput.jpg [ 11639 ]
          Sam Xiao created issue -

            People

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

              Dates

              • Created:
                Updated:
                Resolved: