Closed Bug 590 Opened 27 years ago Closed 26 years ago

List items don't support images as bullets

Categories

(Core :: Layout, defect, P3)

x86
Windows 95
defect

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: NEW → ASSIGNED
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.
Status: RESOLVED → VERIFIED
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.