45888 - center followed by heading 2: text doesn't take on heading 2 formatting

Status: VERIFIED FIXED

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

Description

[ekrock's old account (dead)]

Using Commercial 2000071810 on WinNT 4.0 SP4.

To repro:
1) open new composer window
2) from centering icon on toolbar, select center. I-bar centers as it should.
3) from the HTML Element pulldown on toolbar, choose Heading 2. Heading 2 is
displayed as it should be.
4) type text

Expected: text should be displayed in large H2 formatting

Actual: text is displayed in default Normal formatting

Workaround: select text and re-choose Heading 2.

I think users are likely to hit this as centering the top element on a page and
making it a heading is a very common page design.

Comment 1

[rubydoo123]

assigning to jfrancis for evaluation

Comment 2

[rubydoo123]

setting nsbeta3+

Comment 3

[rubydoo123]

added correctness and nsbeta3+

Comment 4

[rubydoo123]

setting priority in status whiteboard

Comment 5

[leger]

PDT downloading to [nsbeta2-]

Comment 6

[leger]

Meant "PDT downgrading to [nsbeta2-]"

Comment 7

[rubydoo123]

setting to future and adding helpwanted

Comment 8

[Joe Francis]

Not hard to fix.

Comment 9

[Joe Francis]

moz 0.9

Comment 10

[Barrett Lindstrom]

I found some other behaviors that I think stem from this problem as well.

Test case:
1. Open a new document.
2. Type some text as Body text and hit enter.
3. Change type to heading 1.
4. Type some text.

Results:  When you change to heading 1, the cursor is moved back to the end of 
the previous line, however the Style type remains as Heading 1.  But anything 
that is typed will not be in the Heading 1 type, but will be in Body text type. 
 The same workaround as in the beginning of this bug works.

Comment 11

[Joe Francis]

appeasing bug gods

Comment 12

[Joe Francis]

096

Comment 13

[Joe Francis]

Attachment: patch to nsHTMLEditRules.cpp

Comment 14

[Joe Francis]

097

Comment 15

[kinmoz]

Comment on attachment 56021 [details] [diff] [review]
patch to nsHTMLEditRules.cpp

sr=[e-mail]

Comment 16

[Joe Francis]

fix checked in on trunk.

Comment 17

[michael wendell]

Verified on build 2001112803.