Opened 14 years ago
Closed 14 years ago
#7358 closed Bug (fixed)
safari: we can't come out of Numbered/Bulleted list when some formatting is applied to text in list item
Reported by: | Satya Minnekanti | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.5.3 |
Component: | General | Version: | 3.5.3 |
Keywords: | WebKit IBM | Cc: | Damian, Teresa Monahan, James Cunningham |
Description
To reproduce the defect:
- Open Ajax sample, type some text, select it and apply some formatting to it ( for ex: bold).
- Select the text and click on Insert/Remove Numbered list button.
- See that a Numbered list starts with the select text as it's first list item.
- Keep cursor at end of the list item and press Enter twice.
Expected Result:
Cursor comes out of Numbered list and a new empty paragraph starts after Numbered list.
Actual Result:
Cursor will not comes out of Numbered list no matter how many times we press Enter. every time we press Enter it is creating a new empty list item.
Same issue happens with Bulleted list
Attachments (2)
Change History (11)
comment:1 Changed 14 years ago by
Status: | new → confirmed |
---|
comment:2 Changed 14 years ago by
Milestone: | → CKEditor 3.5.3 |
---|
comment:3 Changed 14 years ago by
Keywords: | WebKit added; IBM removed |
---|
comment:4 Changed 14 years ago by
Keywords: | IBM added |
---|
comment:5 Changed 14 years ago by
Owner: | set to Sa'ar Zac Elias |
---|---|
Status: | confirmed → assigned |
Changed 14 years ago by
Attachment: | 7358.patch added |
---|
comment:6 Changed 14 years ago by
Owner: | changed from Sa'ar Zac Elias to Garry Yao |
---|---|
Status: | assigned → review |
comment:7 Changed 14 years ago by
Status: | review → review_failed |
---|
- Type some text, hit CTRL+A, click bold.
- Click on list icon.
- Put cursor at the end of the item.
- Hit ENTER, click increase indent.
- Type some text.
- Hit ENTER twice.
- Type sone text.
Expected: text in the last item should be bold. Actual: text missing the bolden style.
Changed 14 years ago by
Attachment: | 7358_2.patch added |
---|
comment:8 Changed 14 years ago by
Status: | review_failed → review |
---|
You've just caught a small glitch, now use "!CKEDITOR.tools.trim( node.getHtml() )" for empty text container judgment.
comment:9 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | review → closed |
As a tiny issue, I've fixed with [6618] to not defer the release.
Occurs in Safari since revision version [6461]