Bug 9934

Summary: REGRESSION: Selecting text in text field with {line-height:100%} causes it to bounce
Product: WebKit Reporter: David Kilzer (:ddkilzer) <ddkilzer>
Component: FormsAssignee: Dave Hyatt <hyatt>
Status: RESOLVED FIXED    
Severity: Normal CC: adele, daniele.metilli, darin
Priority: P1 Keywords: HasReduction, InRadar, Regression
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
URL: http://subversion.tigris.org/project_issues.html
Attachments:
Description Flags
Test case
none
Patch to not allow the text field's line-height to be smaller than the default. darin: review+

David Kilzer (:ddkilzer)
Reported 2006-07-15 11:15:54 PDT
Steps to reproduce: 1. Open URL: http://subversion.tigris.org/project_issues.html 2. Type a few characters in text field next to "Jump to issue" button in upper-left corner of yellow box. 3. Click-and-drag to highlight characters, but keep holding the mouse down after starting the drag. Expected results: Text in text field should not bounce. Actual results: Text in text field bounces. Notes: Reproduced on locally-built r15456 WebKit with Safari 2.0.4 (419.3) on Mac OS X 10.4.7 (8J135/PowerPC).
Attachments
Test case (46 bytes, text/html)
2006-07-15 11:18 PDT, David Kilzer (:ddkilzer) 		no flags
Details
Patch to not allow the text field's line-height to be smaller than the default. (628 bytes, patch)
2007-01-12 14:48 PST, Dave Hyatt 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
David Kilzer (:ddkilzer)
Comment 1 2006-07-15 11:18:40 PDT
Created attachment 9466 [details] Test case This test case will reproduce the issue as well.
Alice Liu
Comment 2 2006-08-14 14:21:26 PDT
<rdar://problem/4680272>
Dave Hyatt
Comment 3 2007-01-12 14:48:24 PST
Created attachment 12403 [details] Patch to not allow the text field's line-height to be smaller than the default.
Darin Adler
Comment 4 2007-01-12 14:49:08 PST
Comment on attachment 12403 [details] Patch to not allow the text field's line-height to be smaller than the default. r=me; don't forget to put in a layout test
Dave Hyatt
Comment 5 2007-01-12 14:52:16 PST
Fixed in r18816
Dave Hyatt
Comment 6 2007-01-12 15:20:16 PST
*** Bug 12211 has been marked as a duplicate of this bug. ***
Note You need to log in before you can comment on or make changes to this bug.