13095 – Multiple background images not applied to TD elements

RESOLVED FIXED 13095

Multiple background images not applied to TD elements

https://bugs.webkit.org/show_bug.cgi?id=13095

Summary Multiple background images not applied to TD elements

Alex Robinson

Reported 2007-03-16 13:22:41 PDT

Only the first assigned background image in a list of multiple background images is applied to a TD element. The same is also the case with the TABLE and TR elements. On the linked example, two background images on declared on TD but only the first is rendered. The same declaration is also used for BODY where the second image is rendered as expected.

Attachments
Reduced testcase (881 bytes, text/html) 2007-03-17 19:09 PDT, Alex Robinson	no flags	Details
display:table test case (1.95 KB, text/html) 2007-12-14 10:34 PST, Adam van den Hoven	no flags	Details
Oops. ROFL (558 bytes, patch) 2008-01-04 13:42 PST, Dave Hyatt	mitz: review+	Details Formatted Diff Diff
Patch with scope change of paintBackgrounds (1.83 KB, patch) 2008-01-05 05:07 PST, Glan Thomas	glan: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Alex Robinson

Comment 1 2007-03-17 19:09:45 PDT

Created attachment 13684 [details] Reduced testcase

Adam van den Hoven

Comment 2 2007-12-14 10:29:18 PST

Some additional issues: * background-color is not applied in the situation where multiple background images is used * The problem occurs with any elements given display:table-* I've attached a test case showing this.

Adam van den Hoven

Comment 3 2007-12-14 10:34:50 PST

Created attachment 17895 [details] display:table test case Images from iUI (where they're pulled directly).

Dave Hyatt

Comment 4 2007-12-14 11:07:23 PST

Does the problem also occur if the objects are display:block instead?

Alex Robinson

Comment 5 2007-12-14 11:28:36 PST

(In reply to comment #4) > Does the problem also occur if the objects are display:block instead? No. display:block seems to fix it. http://www.fu2k.org/alex/css/bugs/safari-multiple_background_images_on_tables vs http://www.fu2k.org/alex/css/bugs/safari-multiple_background_images_on_tables?displayblock=1

Alex Robinson

Comment 6 2007-12-14 11:43:26 PST

(In reply to comment #5) Ack. Sorry, to be clear, the display:block fixes the problem of both the multiple background images and the background colour as reported in #2

Dave Hyatt

Comment 7 2008-01-04 13:42:38 PST

Created attachment 18274 [details] Oops. ROFL

mitz

Comment 8 2008-01-04 13:43:42 PST

Comment on attachment 18274 [details] Oops. ROFL r=me :-)

Glan Thomas

Comment 9 2008-01-05 05:07:00 PST

Created attachment 18288 [details] Patch with scope change of paintBackgrounds Changed scope of RenderBox::paintBackgrounds from private to protected to allow access by RenderTableCell.cpp Original patch included.

Dave Hyatt

Comment 10 2008-01-07 11:56:14 PST

Yeah I'd already made that change locally.

Dave Hyatt

Comment 11 2008-01-07 13:16:02 PST

Fixed in r29241.

Note You need to log in before you can comment on or make changes to this bug.