36703 - CSS-defined bullets (<LI>) don't appear upon first loading

Status: VERIFIED FIXED

(Core :: Layout, defect, P2)

Description

[Blake Ross]

Build ID: 2000042008

Load Moz and visit the URL above.  Scroll down to where it says "Sluzba (WWW i 
WAP) bezi na vsech adresach: " in the right column of the large table.  Below 
it is an HTML list (using <UL> to start it and <LI> for the bullets).  The same 
type of bulleted lists appear in two gray and yellow columns near the bottom of 
the page.  Near the top of the page in the source is CSS defining the image to 
be used for the bullets (it's a sideways red triange).  However, upon first 
loading the website, I am unable to see these image bullets.  Instead, I see a 
normal list with no bullet/image next to it (which would be the typical result 
if the browser couldn't load the image or the image didn't exist in the 
specified path).  However, if you (or I, at least) reload the page, these 
bullets then show up.

What's going on here?

Comment 1

[Pierre Saslawsky]

The list element should reflow when the image is received. I'm going to attach a 
better testcase. Instead of reloading the page, just resize it to cause a reflow 
of the list. Reassigned to HTML Element.

Comment 2

[Pierre Saslawsky]

Attachment: testcase

Comment 3

[rickg]

Definitely a layout bug.

Comment 4

[buster]

Waqar:  here's a interesting image-related bug.  I'll bet what's happening here 
is the bullet frame isn't properly dealing with the incremental reflow that is 
issued when the image loads the first time.   The second time through, the image 
is in the cache and we get the data back for it immediately, so it works fine.

We need to fix this for proper CSS1 compliance.

Comment 5

[ekrock's old account (dead)]

nsbeta2 6/1-.

Comment 6

[Kevin McCluskey (gone)]

Taking this bug from Waqar

Comment 7

[leger]

Putting on [nsbeta2+][6/15] radar.

Comment 8

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

*** Bug 41169 has been marked as a duplicate of this bug. ***

Comment 9

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

The bug I just marked as a duplicate seems to be the same basic problem,
although perhaps with slightly different symptoms.  In that bug (see simple
testcase attached to it), the images are aligned incorrectly the first time they
load, since (I presume) the dimensions are not known at initial reflow, and the
incremental reflow doesn't reflow the list items.

Comment 10

[Kevin McCluskey (gone)]

Setting Milestone to M17

Comment 11

[Kevin McCluskey (gone)]

I have a fix for this in my tree. The problem was the bullet frame was not 
asked to reflow after the image came in because it does not live within a 
line. The solution was to mark the line dirty instead of the bullet frame 
when the image comes in. 


        // The bullet frame is not in a line list so marking it dirty
        // has no effect. The reflowing of the bullet frame is done 
        // indirectly.

Comment 12

[Kevin McCluskey (gone)]

Changed nsBulletFrame to set the dirty flag on it's parent's first frame instead 
of on itself when it's image is updated. 

Setting the dirty flag on the BulletFrame does not work because the BulletFrame 
is reflowed only when the line which contains
the <LI>'s  text is reflowed. The BulletFrame itself does not reside inside a 
line, so it was not being reflowed when the <LI>'s
image eventually loaded. 

Fixed in 6/8/2000 4:35PM build

Comment 13

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

*** Bug 42177 has been marked as a duplicate of this bug. ***

Comment 14

[Chris Petersen]

Fixed in the June 19th build (2000061908).

Comment 15

[Heikki Toivonen (remove -bugzilla when emailing directly)]

SPAM. HTML Element component deprecated, changing component to Layout. See bug
88132 for details.