Bug 9934 - REGRESSION: Selecting text in text field with {line-height:100%} causes it to bounce
Summary: REGRESSION: Selecting text in text field with {line-height:100%} causes it to...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Forms (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P1 Normal
Assignee: Dave Hyatt
URL: http://subversion.tigris.org/project_...
Keywords: HasReduction, InRadar, Regression
: 12211 (view as bug list)
Depends on:
Blocks:
 
Reported: 2006-07-15 11:15 PDT by David Kilzer (:ddkilzer)
Modified: 2007-01-12 15:20 PST (History)
3 users (show)

See Also:


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+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Kilzer (:ddkilzer) 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).
Comment 1 David Kilzer (:ddkilzer) 2006-07-15 11:18:40 PDT
Created attachment 9466 [details]
Test case

This test case will reproduce the issue as well.
Comment 2 Alice Liu 2006-08-14 14:21:26 PDT
<rdar://problem/4680272>
Comment 3 Dave Hyatt 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.
Comment 4 Darin Adler 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
Comment 5 Dave Hyatt 2007-01-12 14:52:16 PST
Fixed in r18816
Comment 6 Dave Hyatt 2007-01-12 15:20:16 PST
*** Bug 12211 has been marked as a duplicate of this bug. ***