Bug 11951

Summary:

REGRESSION (r10909): PHP Button Misplaced

Product:

WebKit

Reporter:

Pascal Kriete <pascal.kriete>

Component:

Layout and Rendering

Assignee:

Nobody <webkit-unassigned>

Status:

VERIFIED FIXED

Severity:

Normal

CC:

ddkilzer, hyatt, mitz

Priority:

Keywords:

HasReduction, InRadar, Regression

Version:

419.x

Hardware:

Mac

OS:

OS X 10.4

URL:

http://studivz.net

Attachments:

Description	Flags
Test case	none
white-space:pre test case	none
Another test case	none

Pascal Kriete

Reported 2006-12-23 15:05:09 PST

The "Immatrikulieren" button on www.studivz.net is shifted to the right, it should be underneath the 'Einloggen' button. This works fine on the regular Safari. The same happens on the Italian version of the page (www.studiln.net). It makes it very difficult to actually push the button as it can only be hit on the very left where it is not partially covered by the other frame.

Attachments
Test case (529 bytes, text/html) 2006-12-24 06:16 PST, mitz	no flags	Details
white-space:pre test case (226 bytes, text/html) 2006-12-24 07:29 PST, mitz	no flags	Details
Another test case (336 bytes, text/html) 2006-12-24 07:34 PST, mitz	no flags	Details
View All Add attachment proposed patch, testcase, etc.

mitz

Comment 1 2006-12-24 06:16:30 PST

Created attachment 11994 [details] Test case Both cases should have the same layout.

mitz

Comment 2 2006-12-24 07:22:55 PST

Regressed in <http://trac.webkit.org/projects/webkit/changeset/10909> when the 'white-space' property of <input type="button"> was changed. However, even with 'white-space:pre' this behavior seems wrong. That's what needs to be fixed.

mitz

Comment 3 2006-12-24 07:29:31 PST

Created attachment 11995 [details] white-space:pre test case Weird, this show the opposite behavior, but still one of the cases is wrong.

mitz

Comment 4 2006-12-24 07:34:19 PST

Created attachment 11996 [details] Another test case Not doing too well in this case either.

Mark Rowe (bdash)

Comment 5 2007-01-28 15:56:31 PST

<rdar://problem/4960119>

mitz

Comment 6 2007-02-02 10:14:36 PST

ddkilzer, it looks like your latest findNextLineBreak patch fixed this bug!

David Kilzer (:ddkilzer)

Comment 7 2007-02-02 21:34:18 PST

(In reply to comment #6) > ddkilzer, it looks like your latest findNextLineBreak patch fixed this bug! Yay me! See Bug 12490 and Bug 12423. Closing as RESOLVED/FIXED as this works regardless now.

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