230504 - list item disappears when switching stylesheet

Status: NEW

(Core :: Layout: Block and Inline, defect)

Description

[Sekundes]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; rv:1.7a) Gecko/20040108
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; rv:1.7a) Gecko/20040108

list item disappears when switching stylesheet

Reproducible: Always

Steps to Reproduce:
1.Open the testcase.
2.Use style test.
3.The list item disappears.

Comment 1

[Sekundes]

Attachment: test style

Comment 2

[Sekundes]

testcase...
data:text/html;charset=utf-8;base64,PGxpbmsgaHJlZj0iaHR0cDovL2J1Z3ppbGxhLm1vemlsbGEub3JnL2F0dGFjaG1lbnQuY2dpP2lkPTEzODcwMSZhY3Rpb249dmlldyIgcmVsPSJhbHRlcm5hdGUgc3R5bGVzaGVldCIgdGl0bGU9InRlc3QiPg0KPHVsPg0KICA8bGk%2BPGEgaHJlZj0idGVzdC5odG1sIj50ZXN0PC9hPjwvbGk%2BDQo8L3VsPg%3D%3D

Comment 3

[Sekundes]

I got same results in http://quadrone.org.
select basic theme and then select any style again.
The Firebird and Thunderbird links are disappeared.

Comment 4

[Isaac Hwak Han]

Attachment: testcase (javascript version)

Press "Toggle Style".

Comment 5

[Sven Krohlas]

Confirming using Mozilla/5.0 (X11; U; Linux i586; en-US; rv:1.6) Gecko/20040109

Comment 6

[José Jeria]

Not sure if this is the correct component, but definitly not Browser-General

Comment 7

[Boris Zbarsky [:bzbarsky]]

So it looks like dynamic changes to list-style-position are not quite happy...
Looking at the code in nsBlockFrame::UpdateBulletPosition
(http://lxr.mozilla.org/seamonkey/source/layout/html/base/src/nsBlockFrame.cpp#1587),
it looks like the bullet and the first item in the first line both get stuffed
into the first line, which seems incorrect when that first item is a block...

Comment 8

[Boris Zbarsky [:bzbarsky]]

See also bug 36854, which discusses why what we're doing in the non-dynamic case
for list-style-position:inside is wrong too.

Comment 9

[Jonathan]

Using the test case Isaac provided, toggling the style causes the bullet point to jump from the 1st column to the 2nd column above the word "test"

I don't know whether this means this has been fixed or not