ICEfaces
  1. ICEfaces
  2. ICE-4758

inputRichtext: Can't remove style that is applies to text in IE

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.8.1
          • Fix Version/s: 1.8.2-RC1, 1.8.2
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            IE

            Description

            In IE if some text is highlighted and then style is applied via the drop down menu. These styles can't be removed. This isn't a problem in Firefox.

            Sample code:
            <ice:inputRichText height="300px" width="600px"
                                               skin="silver" toolbar="Default"
                                               saveOnSubmit="true"
                                               value="contentField" rendered="true" >
            </ice:inputRichText>

              Activity

              Descending order - Click to sort in ascending order
              Hide
              Permalink
              Joanne Bai added a comment -

              Test result on ICEfaces trunk revision 19228

              • In IE6 and 7 the applied style ("Deleted Text", for example) doesn't get removed
              • Works fine in FF3.5

              Test app committed to repo\qa\trunk\Regression\ICE-4758

              Show
              Joanne Bai added a comment - Test result on ICEfaces trunk revision 19228 In IE6 and 7 the applied style ("Deleted Text", for example) doesn't get removed Works fine in FF3.5 Test app committed to repo\qa\trunk\Regression\ ICE-4758
              Hide
              Permalink
              Adnan Durrani added a comment -

              Following FCK patch applied:
              http://dev.fckeditor.net/changeset/3882

              The following 3 files were involved in the patch. Further patch details can be found in above link.
              editor/_source/internals/fckdomtools.js
              editor/_source/internals/fckxhtml_ie.js
              editor/_source/internals/fckxhtml.js

              Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\inputrichtext\fckeditor.zip
              Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\inputrichtext\fckeditor.zip
              Completed: At revision: 19205

              Show
              Adnan Durrani added a comment - Following FCK patch applied: http://dev.fckeditor.net/changeset/3882 The following 3 files were involved in the patch. Further patch details can be found in above link. editor/_source/internals/fckdomtools.js editor/_source/internals/fckxhtml_ie.js editor/_source/internals/fckxhtml.js Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\inputrichtext\fckeditor.zip Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\inputrichtext\fckeditor.zip Completed: At revision: 19205
              Hide
              Permalink
              Adnan Durrani added a comment -

              After testing FCK API painstakingly, It came across that its due to a call of editor.GetXHTML function. After google I found that this bug was introduced to the FCKEditor since FCKeditor 2.6.3 and seems like its fixed a month ago and targeted to FCKeditor 2.6.5.

              The FCKEditor release is used by ICEFaces is 2.6.4

              http://dev.fckeditor.net/ticket/325
              http://dev.fckeditor.net/ticket/2834
              http://dev.fckeditor.net/ticket/2156

              Show
              Adnan Durrani added a comment - After testing FCK API painstakingly, It came across that its due to a call of editor.GetXHTML function. After google I found that this bug was introduced to the FCKEditor since FCKeditor 2.6.3 and seems like its fixed a month ago and targeted to FCKeditor 2.6.5. The FCKEditor release is used by ICEFaces is 2.6.4 http://dev.fckeditor.net/ticket/325 http://dev.fckeditor.net/ticket/2834 http://dev.fckeditor.net/ticket/2156
              Hide
              Permalink
              Arran Mccullough added a comment -

              I don't see the same behavior on the FCKEditor demo. I have noticed that when its working the added style is highlighted the same as the text (in white). When the added style can't be removed it is in black when the text is in white. This is when everything is highlighted/selected.

              Show
              Arran Mccullough added a comment - I don't see the same behavior on the FCKEditor demo. I have noticed that when its working the added style is highlighted the same as the text (in white). When the added style can't be removed it is in black when the text is in white. This is when everything is highlighted/selected.
              Hide
              Permalink
              Adnan Durrani added a comment -

              Arran can you please apply above steps to the following link unde IE 7:
              http://www.fckeditor.net/demo

              Did you see the same behaviour?

              Show
              Adnan Durrani added a comment - Arran can you please apply above steps to the following link unde IE 7: http://www.fckeditor.net/demo Did you see the same behaviour?
              Hide
              Permalink
              Arran Mccullough added a comment -

              I've only tested with IE7.

              Show
              Arran Mccullough added a comment - I've only tested with IE7.
              Hide
              Permalink
              Adnan Durrani added a comment -

              Is this specific to any version of IE?

              Show
              Adnan Durrani added a comment - Is this specific to any version of IE?
              Hide
              Permalink
              Arran Mccullough added a comment -

              I've attached a test case where I can replicate this issue. Steps to reproduce:

              • Type in some text in the text field.
              • Highlight the text.
              • Change the style for the text, for example: "Deleted Text".
              • Go back and try and remove the "Deleted Text" style, in IE it doesn't get removed.

              Also one thing to note is that I can't replicate this in the Component Showcase.

              Show
              Arran Mccullough added a comment - I've attached a test case where I can replicate this issue. Steps to reproduce: Type in some text in the text field. Highlight the text. Change the style for the text, for example: "Deleted Text". Go back and try and remove the "Deleted Text" style, in IE it doesn't get removed. Also one thing to note is that I can't replicate this in the Component Showcase.
              Hide
              Permalink
              Adnan Durrani added a comment -

              I am not able to reproduce it, can you please post some steps and screen shots?

              Show
              Adnan Durrani added a comment - I am not able to reproduce it, can you please post some steps and screen shots?

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: