Bug 27656

Summary: REGRESSION: Clicking on popup menu changes layout
Product: WebKit Reporter: Ton Voon <ton.voon>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: krinklemail, robert, simon.fraser
Priority: P1 Keywords: HasReduction, InRadar, Regression
Version: 528+ (Nightly build)   
Hardware: Mac (Intel)   
OS: OS X 10.5   
Attachments:
Description Flags
Reduced HTML showing problem
none
Movie of problem none

Ton Voon
Reported 2009-07-24 08:44:12 PDT
Created attachment 33451 [details] Reduced HTML showing problem Hi! I have a problem on our web application which uses table-less forms. If you press the popup menu button, the layout changes. Specifically, the floating label on the left hand side becomes indented. This is fully reproduceable. Tests on Safari 3, FF 3.0.12 and FF 3.5 and IE6 show no such problem. This occurs in Safari 4.0.2 and the latest nightly build (WebKit r46282). I've attached a reduced HTML file. I've also attached a movie demonstrating the problem. If I change the label.fields padding-bottom to 3px, then the problem disappears. However, I suspect this is still bad behaviour on Webkit as the initial layout is correct. Ton
Attachments
Reduced HTML showing problem (1.07 KB, text/html)
2009-07-24 08:44 PDT, Ton Voon
no flags
Movie of problem (79.34 KB, video/quicktime)
2009-07-24 08:49 PDT, Ton Voon
no flags
Ton Voon
Comment 1 2009-07-24 08:49:11 PDT
Created attachment 33452 [details] Movie of problem
Alexey Proskuryakov
Comment 2 2009-07-24 22:42:28 PDT
Simon Fraser (smfr)
Comment 3 2009-08-11 17:44:30 PDT
It's focus, not clicking that is the issue; the focus ring is affecting layout.
Ton Voon
Comment 4 2009-08-11 23:13:29 PDT
(In reply to comment #3) > It's focus, not clicking that is the issue; the focus ring is affecting layout. I don't think it is the focus ring (unless there is something happening in the background). If I add this to the CSS: input[type=text]:focus, input[type=password]:focus, textarea:focus, select:focus { outline: 0 none; } This removes the blue ring, but the layout still changes on click.
Ton Voon
Comment 5 2009-08-11 23:48:17 PDT
A comment at doctype.com (http://doctype.com/questions/whats-best-way-overcome-safari-4-bug-css-tableless-form) suggests that a partial redraw could be occuring which is missing the <br> tag from the field prior to the popup. This appears to be correct when I run Quartz Debug. On Safari 3, the redraw flash appears over the previous text box, the popup and the button, whereas in webkit nightly, the redraw flashes for the popup and the button. It seems to me like the previous <br> tag is not considered for redrawing the layout on the popup click, hence missing the clear:left and the label floats are then aligned together.
Alexey Proskuryakov
Comment 6 2011-02-02 11:19:24 PST
*** Bug 53387 has been marked as a duplicate of this bug. ***
Robert Hogan
Comment 7 2012-10-03 12:34:16 PDT
I cannot recreate this on Chromium20. I can however recreate bug 53387 and 18939, so this seems to be a different issue that is resolved.
Robert Hogan
Comment 8 2013-05-02 12:07:14 PDT
This is no longer reproducible on trunk so I'm closing it.
Note You need to log in before you can comment on or make changes to this bug.