#2768: plug-in:basicstyle all basic styles unable to apply reversely
-------------------------------+--------------------------------------------
Reporter: garry.yao | Owner: garry.yao
Type: Bug | Status: assigned
Priority: Normal | Milestone: CKEditor 3.0
Component: Core : Styles | Version: SVN
Keywords: Confirmed Review- |
-------------------------------+--------------------------------------------
Changes (by martinkou):
* keywords: Confirmed Review? => Confirmed Review-
Comment:
There's still one case where the patch does not work:
4. Test removal of a style across two inline tags.
* Original document
{{{
<p>
This is some <strong>sample <em>text</em></strong><em>. You</em>
are using <a href="http://www.fckeditor.net/";>FCKeditor</a>.</p>
}}}
* Press the I button to remove the italics.
* Expected result
{{{
<p>
This is some <strong>sample text</strong>. You are using <a
href="http://www.fckeditor.net/";>FCKeditor</a>.</p>
}}}
* Actual result
{{{
<p>
This is some <strong>sample </strong><strong>text</strong>. You
are using <a href="http://www.fckeditor.net/";>FCKeditor</a>.</p>
}}}
The patch also has some coding style problems. I've talked to Garry on
Skype about them.
--
Ticket URL: <http://dev.fckeditor.net/ticket/2768#comment:8>
FCKeditor <http://www.fckeditor.net/>
The text editor for Internet
------------------------------------------------------------------------------
This SF.net email is sponsored by:
SourcForge Community
SourceForge wants to tell your story.
http://p.sf.net/sfu/sf-spreadtheword
_______________________________________________
FCKeditor-Trac mailing list
FCKeditor-Trac@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac
