Regression:Text field height is incorrect with line-height and ignored percent height
Reported by
adha...@etouch.net,
Apr 28 2016
|
||||||||||
Issue descriptionChrome Version:52.0.2718.0 (Official Build0(64-bit)fd0a0f9879fc02e65d477f57f8f929fa4d7f0b75-refs/heads/master@{#389938}(32/64-bit) OS: Mac OS X(10.10.5, 10.11.4), Windows (7,8,8.1,10),Linux (14.04 LTS) What steps will reproduce the problem? (1)Launch chrome and navigate to http://cbslocal.com/ (2)Click in the search box at the top R.H.S and observe the cursor. Actual:Cursor height is seen different(increased). Expected:Cursor height should not be different. This is a Regression issue broken in M-50 ,below is the Narrow bisect info: https://chromium.googlesource.com/chromium/src/+log/5c174785589ff4d446671eae01bbeb3a045b04c8..deb2b1a6835b09ad4617131a70dab9092a240f31?pretty=fuller&n=10000 Suspecting:r370036? Good build:49.0.2623.0 Bad build:50.0.2625.0 Kindly help to re-assign if your change is not the cause for this issue.
,
Jul 13 2016
,
Oct 12 2016
,
Oct 12 2016
,
Oct 26 2016
,
Oct 26 2016
,
Nov 4 2016
,
Nov 4 2016
Here is a test case: https://jsfiddle.net/cubix/dopceopn/ https://codereview.chromium.org/2478483003
,
Nov 8 2016
,
Jan 18 2017
Caret height should be font-height of character before caret.
,
Jan 18 2017
https://jsfiddle.net/cubix/dopceopn/ demonstrates INPUT box height issue in addition to the caret height issue. We should address the INPUT box height issue.
,
Jan 18 2017
Test result with this suggestion: https://codereview.chromium.org/2478483003/#msg61
,
Aug 4 2017
joone.hur@, do you still want to work on this issue?
,
Aug 5 2017
Yes, I will look into this issue again. |
||||||||||
►
Sign in to add a comment |
||||||||||
Comment 1 by glider@chromium.org
, Jun 16 2016Status: Untriaged (was: Assigned)