RESOLVED FIXED28718
REGRESSION: Images May Disappear if inside an element that is position:relative and display:inline
https://bugs.webkit.org/show_bug.cgi?id=28718
Summary REGRESSION: Images May Disappear if inside an element that is position:relati...
Joseph Pecoraro
Reported 2009-08-25 14:33:06 PDT
Created attachment 38570 [details] Resize the Browser to be Shorter then the Image and it disappears If an image is itself position:relative or contained inside an element that is position:relative it may not paint if the browser's view port doesn't contain the full image. This can cause weird painting issues on a regular webpage such as http://www.creativenotice.com/html-5-humor View of the attached regression in Safari 4.0.3 (Expected Behavior): http://tr.im/x6r4?grabup View of the attached reduction in WebKit r47686 (Bad Behavior): http://tr.im/x6qV?grabup
Attachments
Resize the Browser to be Shorter then the Image and it disappears (1.86 KB, text/html)
2009-08-25 14:33 PDT, Joseph Pecoraro
no flags
Patch (2.73 KB, patch)
2009-08-26 13:30 PDT, Dave Hyatt
hyatt: review+
mitz
Comment 1 2009-08-25 14:38:45 PDT
Joseph Pecoraro
Comment 2 2009-08-25 14:45:51 PDT
Refined even further, it seems as though it may also be specific to an <a> tag. My latest tests showed that: Displays Correctly: <div style="position:relative"> <img ... /> </div> Displays Incorrectly: <a style="position:relative"> <img ... /> </a>
Joseph Pecoraro
Comment 3 2009-08-25 14:50:40 PDT
Arg, spoke too soon! It fails in a span, and so I tried display:inline and indeed this fails too: <div style="display:inline;position:relative"> <img ... /> </div> "inline-block" and "block" both worked correctly, so hopefully this is specific to inline.
Dave Hyatt
Comment 4 2009-08-26 13:30:37 PDT
Simon Fraser (smfr)
Comment 5 2009-08-26 13:44:10 PDT
Is bug 28700 the same issue?
Dave Hyatt
Comment 6 2009-08-26 13:59:13 PDT
No, that bug actually occurs on Snow Leopard.
Eric Seidel (no email)
Comment 7 2009-08-26 14:18:55 PDT
Comment on attachment 38636 [details] Patch Can't we make a test for this? It seems the test is more important than the code change in this sort of bug.
Dave Hyatt
Comment 8 2009-08-26 14:50:00 PDT
Comment on attachment 38636 [details] Patch Dan reviewed this.
Dave Hyatt
Comment 9 2009-08-26 15:34:06 PDT
Fixed in r47798.
Note You need to log in before you can comment on or make changes to this bug.