Opened 14 years ago
Closed 14 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 14 years ago by
Milestone: | → CKEditor 3.5.3 |
---|---|
Status: | new → confirmed |
Version: | → 3.5.3 (SVN - trunk) |
Changed 14 years ago by
Attachment: | 7294.patch added |
---|
comment:2 Changed 14 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 14 years ago by
Status: | review → review_failed |
---|
"currentStyle" is not a cross-browser property.
Changed 14 years ago by
Attachment: | 7294_2.patch added |
---|
comment:5 Changed 14 years ago by
Status: | review_failed → review |
---|
comment:6 Changed 14 years ago by
comment:7 Changed 14 years ago by
Status: | review → review_passed |
---|
Instead of having "isInDoc( el, doc )" I think we could safely have "isOffline( el )".
comment:8 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [6551].
Regression of [6478].