#2837 closed Bug (fixed)
Basicstyle plugin bug
Reported by: | Artur Formella | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.0 |
Component: | General | Version: | SVN (FCKeditor) - Retired |
Keywords: | IE Confirmed Review+ | Cc: |
Description
Steps to reproduce:
-Open trunk/_samples/sample.html?sample=replacebyclass in IE
-Select "some" and click Bold.
-Click Source.
-Back to WYSIWYG.
-Select "using" and click Bold.
Result: JS error. Source code:
<p _cke_expando="65"> This is <strong _cke_expando="70">some sample text</strong>. You are <span _cke_expando="78" _fck_bookmark="1" style="DISPLAY: none"> </span>using <span _cke_expando="78" _fck_bookmark="1" style="DISPLAY: none"> </span><a _cke_expando="79" href="http://www.fckeditor.net/">FCKeditor</a>.</p>
In FF works fine.
Attachments (2)
Change History (12)
comment:1 Changed 14 years ago by
Keywords: | Confirmed added |
---|
comment:2 Changed 14 years ago by
comment:3 Changed 14 years ago by
Owner: | set to Garry Yao |
---|---|
Status: | new → assigned |
Changed 14 years ago by
Attachment: | 2837.patch added |
---|
comment:4 Changed 14 years ago by
Keywords: | Review? added |
---|
Disable caching style element which cause error when content dom is reloaded, it shouldn't have great impact on PF.
comment:5 Changed 14 years ago by
Keywords: | Review- added; Review? removed |
---|
- If the patch is proposing not using anymore a cache, so style._.element is to be removed. There is no sense having it anymore.
- Why line 549 has been moved to 541 (before the if)?
- (Detail) A blank line remained at line 532.
Changed 14 years ago by
Attachment: | 2837_2.patch added |
---|
comment:6 Changed 14 years ago by
Keywords: | Review? added; Review- removed |
---|
comment:7 Changed 14 years ago by
Keywords: | Review+ added; Review? removed |
---|
comment:9 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
The commit should match the patch, which is not true in this case. An unexpected ignore rule has been added to the root. We should avoid such things.
In Opera 9.63 selected text disappears.