Opened 15 years ago
Closed 14 years ago
#4790 closed Bug (fixed)
CKEditor in enterMode = BR, problem with bullet list
Reported by: | Wiktor Walc | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.4.3 |
Component: | Core : Lists | Version: | 3.0 |
Keywords: | Opera | Cc: |
Description
This ticket is similar to #3836. I think it happens only in Opera (tested on 10.10).
- Open the replace by code example page with enterMode = BR;
- Make the content and selection as below:
line1 <ul><li>[item1</li><li>item2]</li></ul> line2
- Click on 'Bullet List' to remove list;
- Expected Result:
line1<br /> item1<br /> item2<br /> line2
- Actual Result:
line1<br /> item1item2<br /> line2
Attachments (1)
Change History (12)
comment:1 Changed 15 years ago by
Component: | General → Core : Lists |
---|
comment:2 Changed 15 years ago by
Milestone: | → CKEditor 3.x |
---|
comment:3 Changed 14 years ago by
Milestone: | CKEditor 3.x |
---|
comment:4 Changed 14 years ago by
Summary: | [Opera] → CKEditor in enterMode = BR, problem with bullet list |
---|
Changed 14 years ago by
Attachment: | 4790.patch added |
---|
comment:5 Changed 14 years ago by
Milestone: | → CKEditor 3.5 |
---|---|
Owner: | set to Garry Yao |
Status: | confirmed → review |
comment:6 Changed 14 years ago by
Milestone: | CKEditor 3.5 → CKeditor 3.4.3 |
---|
comment:7 follow-up: 8 Changed 14 years ago by
It works well for me in Opera and others, although i'm thinking why not to change the appendBogus method which appends an empty text node for Opera only.
comment:8 Changed 14 years ago by
Replying to tobiasz.cudnik:
It works well for me in Opera and others, although i'm thinking why not to change the appendBogus method which appends an empty text node for Opera only.
Do you mean having the entire fix there?
comment:9 Changed 14 years ago by
I'm ok with fixing only appendBogus, while Fred had told me before for "some reason" that we kept using empty space as bogus for Opera.
comment:10 Changed 14 years ago by
Status: | review → review_passed |
---|
I'm not getting same results after patching just appendBogus, for first linebreak you have to hit Enter twice.
So R+ for the 4790 patch.
comment:11 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [6112].
Milestone CKEditor 3.x deleted