ICEfaces
  1. ICEfaces
  2. ICE-5995

Can't remove styling from inputRichText with saveOnSubmit=true 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, 2.0.0
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      IE

      Description

      When an inputRichText component has saveOnSubmit=true, adding style (Bold, Underline, Italics) to the text can't be reverted back once the style is applied.

      Steps to reproduce issue:
       - Type in some text into the text field.
       - Highlight the text.
       - Select the "B" to bold the selected text.
       - De-highlight the text.
       - Type in some more text.
       - Select all of the text in the field.
       - The "B" is highlighted but can't be turned off.

      This can be reproduced with the Italic Style button and the Underline style button. This can only be reproduced with saveOnSubmit=true and in IE.

      Sample code:

      <ice:inputRichText value="#{testBean.text}"
                                                     toolbar="Default"
                                                     saveOnSubmit="true"/>

        Activity

        Arran Mccullough created issue -
        Arran Mccullough made changes -
        Field Original Value New Value
        Salesforce Case [5007000000D4xVb]
        Arran Mccullough made changes -
        Fix Version/s 1.8.2-EE-GA_P02 [ 10226 ]
        Hide
        Ken Fyten added a comment -

        Please retest using latest icefaces/trunk.

        Show
        Ken Fyten added a comment - Please retest using latest icefaces/trunk.
        Ken Fyten made changes -
        Assignee Mandeep Hayher [ mandeep.hayher ]
        Hide
        Mandeep Hayher added a comment -

        Issues is reproducible on icefaces trunk revision# 22992, Tomcat6 & IE8

        Show
        Mandeep Hayher added a comment - Issues is reproducible on icefaces trunk revision# 22992, Tomcat6 & IE8
        Mandeep Hayher made changes -
        Assignee Mandeep Hayher [ mandeep.hayher ] Ken Fyten [ ken.fyten ]
        Ken Fyten made changes -
        Assignee Priority P2
        Assignee Ken Fyten [ ken.fyten ] Adnan Durrani [ adnan.durrani ]
        Adnan Durrani made changes -
        Assignee Adnan Durrani [ adnan.durrani ] Mandeep Hayher [ mandeep.hayher ]
        Hide
        Mandeep Hayher added a comment -

        Issue is still reproducible on Icefaces trunk revision# 23027, Tomcat6 & IE8.

        Show
        Mandeep Hayher added a comment - Issue is still reproducible on Icefaces trunk revision# 23027, Tomcat6 & IE8.
        Mandeep Hayher made changes -
        Assignee Mandeep Hayher [ mandeep.hayher ] Adnan Durrani [ adnan.durrani ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #23049 Sat Nov 06 14:32:42 MDT 2010 adnan.durrani Fix for ICE-5995 (Can't remove styling from inputRichText with saveOnSubmit=true in IE)
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/inputrichtext/fckeditor_ext.js
        Adnan Durrani made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Adnan Durrani added a comment -

        This FCK bug was fixed under this case, which was related to getData()
        http://dev.fckeditor.net/changeset/3882

        We did applied that patch already, and it is part of new FCK release 2.6.6. But there are still some issue with getXHTML() call.

        So instead of making getXHTML() call, we are extracting contents form the markup.

        Fix allied to trunk:

        Command: Commit
        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\fckeditor_ext.js
        Completed: At revision: 23049

        Show
        Adnan Durrani added a comment - This FCK bug was fixed under this case, which was related to getData() http://dev.fckeditor.net/changeset/3882 We did applied that patch already, and it is part of new FCK release 2.6.6. But there are still some issue with getXHTML() call. So instead of making getXHTML() call, we are extracting contents form the markup. Fix allied to trunk: Command: Commit 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\fckeditor_ext.js Completed: At revision: 23049
        Adnan Durrani made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Ken Fyten added a comment -

        Fix needs to be applied also the ICEfaces 2 compat variant of inputRichText.

        Show
        Ken Fyten added a comment - Fix needs to be applied also the ICEfaces 2 compat variant of inputRichText.
        Ken Fyten made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Ken Fyten made changes -
        Fix Version/s 2.0.0 [ 10230 ]
        Fix Version/s 1.8.3 [ 10211 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #23059 Mon Nov 08 22:46:09 MST 2010 adnan.durrani Fix for ICE-5995 (Can't remove styling from inputRichText with saveOnSubmit=true in IE)
        Files Changed
        Commit graph MODIFY /icefaces2/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/inputrichtext/fckeditor_ext.js
        Hide
        Adnan Durrani added a comment -

        Patch applied to compat

        Show
        Adnan Durrani added a comment - Patch applied to compat
        Adnan Durrani made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Priority P2

          People

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

            Dates

            • Created:
              Updated:
              Resolved: