Opened 15 years ago
Closed 15 years ago
#7294 closed Bug (fixed)
IE we can't convert Numbered list to a Bulleted list and vice versa
| Reported by: | Satya Minnekanti | Owned by: | Sa'ar Zac Elias |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.5.3 |
| Component: | General | Version: | 3.5.3 |
| Keywords: | IBM IE | Cc: | Damian, James Cunningham, Teresa Monahan |
Description
To reproduce the defect
- Open CK Editor and type 2 paragraphs of text.
- select 2 paragraphs and click on Insert/Remove Numbered list button
- see that a Numbered list appears with 2 list items.
- Now select both list items and click on Insert/Remove Bulleted list button.
Expected Result:
Numbered list is converted to a Bulleted list.
Actual Result:
Nothing happens and we are seeing an error in IE and error says 'this.$currentStyle' is null or not an object.
same issue occurs when we try to convert a Bulleted list to a Numbered list.
Attachments (2)
Change History (10)
comment:1 Changed 15 years ago by
| Milestone: | → CKEditor 3.5.3 |
|---|---|
| Status: | new → confirmed |
| Version: | → 3.5.3 (SVN - trunk) |
Changed 15 years ago by
| Attachment: | 7294.patch added |
|---|
comment:2 Changed 15 years ago by
| Keywords: | IE added |
|---|---|
| Owner: | set to Sa'ar Zac Elias |
| Status: | confirmed → review |
IE will generate an error when getting current styles from elements that are yet to be in the DOM tree.
comment:4 Changed 15 years ago by
| Status: | review → review_failed |
|---|
"currentStyle" is not a cross-browser property.
Changed 15 years ago by
| Attachment: | 7294_2.patch added |
|---|
comment:5 Changed 15 years ago by
| Status: | review_failed → review |
|---|
comment:6 Changed 15 years ago by
comment:7 Changed 15 years ago by
| Status: | review → review_passed |
|---|
Instead of having "isInDoc( el, doc )" I think we could safely have "isOffline( el )".
comment:8 Changed 15 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [6551].

Regression of [6478].