VERIFIED FIXED 6301
REGRESSION: Border repaint glitch
https://bugs.webkit.org/show_bug.cgi?id=6301
Summary REGRESSION: Border repaint glitch
mitz
Reported 2005-12-30 04:14:56 PST
Summary: fieldsets don't repaint completely when their size changes, so parts of their border are either missing or left behind. To reproduce: open the testcase in Safari. Click each Test button once. Expected: the bottom border to be painted correctly after resize. Actual: in the top case (shrinking), a segment is missing on each side of the bottom border; in the bottom case (growing), a segment from each side of the bottom border remains behind.
Attachments
testcase (880 bytes, text/html)
2005-12-30 04:15 PST, mitz
no flags
Repaint moving border (4.51 KB, patch)
2005-12-30 05:37 PST, mitz
darin: review+
mitz
Comment 1 2005-12-30 04:15:30 PST
Created attachment 5377 [details] testcase
mitz
Comment 2 2005-12-30 04:21:16 PST
Actually, this is a problem with borders in general, a DIV with solid border works just as well in the testcase.
mitz
Comment 3 2005-12-30 05:37:01 PST
Created attachment 5380 [details] Repaint moving border Like with bug 5699, an automatic test is currently impossible.
Darin Adler
Comment 4 2005-12-30 11:30:39 PST
Comment on attachment 5380 [details] Repaint moving border What about borderLeft/Top?
mitz
Comment 5 2005-12-30 11:44:16 PST
(In reply to comment #4) > What about borderLeft/Top? The left/top borders don't move so there's no need to repaint them. You're in this case: // We didn't move, but we did change size. Invalidate the delta, which will consist of possibly // two rectangles (but typically only one). You only need to extend this to include the area that was/will be occupied by the border that moves.
Darin Adler
Comment 6 2005-12-30 13:30:40 PST
Comment on attachment 5380 [details] Repaint moving border r=me
Joost de Valk (AlthA)
Comment 7 2006-01-22 04:56:50 PST
Removing keyword(s) since bug is fixed.
Joost de Valk (AlthA)
Comment 8 2006-01-22 05:01:12 PST
Removing keyword(s) since bug is fixed.
Eric Seidel (no email)
Comment 9 2006-01-31 21:20:52 PST
Removing Regression keyword from bugs already fixed.
Note You need to log in before you can comment on or make changes to this bug.