Changes between Version 1 and Version 3 of Ticket #11802
- Timestamp:
- Apr 10, 2014, 5:45:31 PM (11 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #11802
-
Property
Status
changed from
newtoconfirmed -
Property
Version
changed from
4.1to -
Property
Summary
changed from
Switching to source mode modify source on indented listtoMargin is set on list item when creating list from indented paragraph
-
Property
Status
changed from
-
Ticket #11802 – Description
v1 v3 3 3 3. Click few times "Increase indent" 4 4 4. Click "Insert/Remove ordered list" 5 5. Type some text 6 5. Click "Source" to get into source mode 7 As expected source is: 8 <ol> 9 <li style="margin-left: 80px;">Hidden message</li> 10 </ol> 11 7. Repeat step 5 twice. 12 Actual result: 13 <ol> 14 <li>Hidden message</li> 15 </ol> 5 5. Switch to source mode: 16 6 17 Cannot reproduce in version 4.1.0 because button "Increase indent" is disabled. 18 In version 4.0 everything works as expected. 7 {{{ 8 <ul> 9 <li style="margin-left: 120px;">Foo</li> 10 <li>Bar</li> 11 </ul> 12 }}} 13 14 Expected: 15 16 {{{ 17 <ul style="margin-left: 120px;"> 18 <li>Foo</li> 19 <li>Bar</li> 20 </ul> 21 }}} 22 23 #Reasoning 24 25 Paragraph's margin should be moved to `ol/li` when creating list, because this is where indent/outdent buttons add/remove margin when you have a caret in the first item of a list. 26 27 However, I'm not sure what if we're creating list out of few paragraphs when each have different margin. I think that in such case it's best to remove those margins and "normalize" the situation. Otherwise, we'd have to go crazy and e.g. create sublists based on indentation of following paragraphs, but that would be a waste of time and we don't know if user wanted to do that anyway. 28 29 Alternatively, we can simply remove all margins when creating list, because none of the solutions seem to be 100% correct. Everyone can have different idea about how that should work.
