Opened 14 years ago
Closed 14 years ago
#7092 closed Bug (fixed)
Nested inline styling
Reported by: | Frederico Caldeira Knabben | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.5.3 |
Component: | Core : Styles | Version: | 3.3.2 |
Keywords: | Cc: |
Description (last modified by )
- Load the following in the source view (important to be in source view):
<p> <strong>Test</strong></p>
- Back to wysiwyg, put the caret at the end of the line with mouse click.
- Type anything, like "aaa".
- Click the Bold button.
Several strange things happen:
- The elements path will show a nested <strong>.
- If you continue typing the text will be bold.
- Even if the text is bold, the toolbar button is not "on" (blue).
Introduced with the 3.3.2. Works well with the 3.3.1.
Confirmed with FF at least.
Attachments (2)
Change History (10)
comment:1 Changed 14 years ago by
Description: | modified (diff) |
---|
comment:2 Changed 14 years ago by
Hum, it looks like the above TC will still not be reproducible always.
The following looks like more reliable:
- Load the editor.
- Click the "New Page" button.
- Type "Test".
- CTRL+A
- Click Bold.
- Go to Source and back to WYSIWYG.
- Click to put the cursor at the end of the text.
- Type 'aaa'.
- Click Bold.
comment:3 Changed 14 years ago by
Status: | new → confirmed |
---|
Changed 14 years ago by
Attachment: | 7092.patch added |
---|
comment:5 Changed 14 years ago by
Owner: | set to Sa'ar Zac Elias |
---|---|
Status: | confirmed → review |
Changed 14 years ago by
Attachment: | 7092_2.patch added |
---|
comment:6 Changed 14 years ago by
Owner: | changed from Sa'ar Zac Elias to Garry Yao |
---|
Isn't it just safe to remove the the cache, as it's mainly for performance consideration while same thing has already done at CKEDITOR.command::setState.
comment:7 Changed 14 years ago by
Status: | review → review_passed |
---|
comment:8 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [6432].
I've originally reported it with the following HTML to be loaded:
In this case it works well. So, it's important to have the line break after the opener <p>.