45792 - indenting a roman list goes to numbered list

Status: VERIFIED WORKSFORME

(Core :: DOM: Editor, defect, P4)

Description

[sujay]

using 7/18 build of netscape

1) launch netscape
2) launch composer
3) make a numbered list
4) go into Format | List Props
5) change to Roman numberal lower case
6) enter more list items
7) indent the list item

notice it indents into a regular numbered list...shouldn't it
indent into a the roman numeral lower case..

see 4.x behavior...

all platforms.

Comment 1

[rubydoo123]

we had this agruemnt in the past with Kipp, he insisted that the list would 
default to the 'default' list type -- which I totally disagree with, the 
indentations should alter the list styles IMHO. So, assigning this over to marc 
for evaluation -- Marc, you are the CSS guy right?

Comment 2

[Marc Attinasi]

The problem is that a new <OL> is being created for the indented item, but the 
type is not being propagated from the parent <OL>. After indenting one of the 
list items here is the HTML that is produced:

<ol type="I">
  <li>one</li>
  <li>two</li>
  <li>three</li>
  <li>four</li>
  <ol>
    <li>five</li>
  </ol>
  <li>six</li>
  <li>seven<br>
  </li>
</ol>

Notice that the top <OL> has type="I" but that the inner <OL> after item 'four' 
is not given the type attribute. If you want the indented list to have the same 
type you have to propagate it when you create the new <OL>, and that is 
something the editor has to do (style has nothing to do with the HTML 
generation).

NOTE: you could just give the inner <OL> a style attribute of 
style="list-style-type:inherit" if you cannot track the actual type of the 
parent:

<ol type="I">
  <li>one</li>
  <li>two</li>
  <li>three</li>
  <li>four</li>
  <ol style="list-style-type:inherit">
    <li>five</li>
    <ol style="list-style-type:inherit">
      <li>XXX</li>
    </ol>
  </ol>
  <li>six</li>
  <li>seven<br>
  </li>
</ol>

(for example).

Comment 3

[rubydoo123]

assigning to jfrancis. Joe, the indented list needs to be within the <LI></LI>, 
not outside

Comment 4

[rubydoo123]

really assigning to Joe this time

Comment 5

[rubydoo123]

added correctness and nsbeta3+

Comment 6

[rubydoo123]

setting priority in status whiteboard

Comment 7

[Joe Francis]

accepting bug

Comment 8

[leger]

Marking as WorksForMe.  This bug does not occur on 2000-08-24-04 build.  Sujay, 
can you check all platforms with latest build and verify this please?

Comment 9

[sujay]

verified on all platforms.