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

        Hide
        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?
        Hide
        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
        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
        Arran Mccullough added a comment -

        I've only tested with IE7.

        Show
        Arran Mccullough added a comment - I've only tested with IE7.
        Hide
        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
        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
        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
        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
        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

          People

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

            Dates

            • Created:
              Updated:
              Resolved: