#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 15 years ago by
Keywords: | Confirmed added |
---|
comment:2 Changed 15 years ago by
comment:3 Changed 15 years ago by
Owner: | set to Garry Yao |
---|---|
Status: | new → assigned |
Changed 15 years ago by
Attachment: | 2837.patch added |
---|
comment:4 Changed 15 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 15 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 15 years ago by
Attachment: | 2837_2.patch added |
---|
comment:6 Changed 15 years ago by
Keywords: | Review? added; Review- removed |
---|
comment:7 Changed 15 years ago by
Keywords: | Review+ added; Review? removed |
---|
comment:9 Changed 15 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.