Opened 14 years ago
Closed 13 years ago
#7221 closed Bug (fixed)
A click on a horizontal list's element makes it become a vertical list
Reported by: | guillaumesmo | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.5.4 |
Component: | Core : Styles | Version: | 3.5.1 |
Keywords: | IBM | Cc: | satya_minnekanti@… |
Description
When you have a horizontal list (which is a list with list-style-type:none and li elements display:inline) and you click on a list element, the element goes to the next line as if he becomes a vertical list element. Going to source view and back to wysiwyg view makes the list horizontal again.
Paste the following code in the source view of a ckeditor, and then go back to wysiwyg view. Now click on a "lol" to reproduce the bug.
<style> ul.list1{margin:20px 0px 0px 0px; padding:0px; list-style-type:none;} ul.list1 li{display:inline; padding:20px;} </style> <ul class="list1"> <li> lol</li> <li> lol</li> <li> lol</li> <li> lol</li> <li> lol</li> <li> lol</li> </ul>
I'm using Mac OS X Snow Leopard with FireFox 3.5.16
Attachments (2)
Change History (9)
comment:1 Changed 14 years ago by
Component: | General → Core : Styles |
---|---|
Status: | new → confirmed |
Version: | → 3.5.1 |
comment:2 Changed 14 years ago by
Cc: | satya_minnekanti@… added |
---|---|
Keywords: | IBM added |
Changed 13 years ago by
Attachment: | 7221.patch added |
---|
comment:3 Changed 13 years ago by
Owner: | set to Garry Yao |
---|---|
Status: | confirmed → review |
comment:4 Changed 13 years ago by
Status: | review → review_failed |
---|
There is good part of the code that depends on the current behavior of the elements path class. Because of this, a few strange things started to happen, after patch:
- The elements path bar "body > ul > p > li" when inside a list item.
- By hitting ENTER at the end of the list, the elements path bar shows "body > ul > p". If you then type some text and go to source, the text moves before the list.
comment:5 Changed 13 years ago by
Status: | review_failed → review |
---|
That's true, actually the ticket tc is a boundary pusher, as by our design dtd semantics are first-class citizens of behavior guide thus making the editor work with display-mangled structure (which is commonly happened when using the editor as an design tool instead of content composer) will not be our priority, so a simple fix that just make editor respect user content would be enough here.
Changed 13 years ago by
Attachment: | 7221_2.patch added |
---|
comment:6 Changed 13 years ago by
Milestone: | → CKEditor 3.5.4 |
---|---|
Status: | review → review_passed |
comment:7 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [6816].
Be styling proof on path block evaluation.