Closed
Bug 590
Opened 27 years ago
Closed 26 years ago
List items don't support images as bullets
Categories
(Core :: Layout, defect, P3)
Tracking
()
VERIFIED
FIXED
People
(Reporter: angus, Assigned: buster)
References
()
Details
See URL for test case and sample code. I'm guessing that kipp is the right
person to update the LI element to handle images for bullets, and peterl can
hook up style to this.
1) It triggered a crash in the string code (fixed)
2) It doesn't parse right (not fixed)
3) The image bullets don't always show up properly (not yet looked at)
The images work for me AFTER I've triggered an incremental reflow, therefore
this bug is really an incremental reflow bug. And there are two of them:
1) the first list doesn't reflow properly when the image size arrives
2) the 3rd list doesn't either and it's in a table; you have to work hard to
get the table to reflow.
I'll look into this more and determine where the incremental reflow is going
awry.
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Component: Unknown → Layout
Resolution: --- → FIXED
I've modified the frame-image-loader API's and implementation so that a
callback can be specified. I've hooked it up (for now) to just the case
where a reflow is needed. The previous code automatically took the
frame, found it's content and then triggered a content-changed reflow.
All well and good *if the frame had content*. Well, image bullets don't
have content anymore so they weren't getting notified when the image had
been loaded.
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
Comment 4•26 years ago
|
||
Verified fixed using 1.28.99 build (on Mac OS, Win 32, Linux.)
I do note that the images on the test page on Mac & Linux are stacked on top of
each other vertically; i.e. the 3-4 pixel row of white space that occurs on the
Win32 build [and IE] doesn't take place on these platforms.
Will probably split into a separate bug and write up, unless already written up.
You need to log in
before you can comment on or make changes to this bug.
Description
•