<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>137269</bug_id>
          
          <creation_ts>2014-09-30 14:24:29 -0700</creation_ts>
          <short_desc>weird input[type=number] incr/decrement button behavior at odd-numbered font sizes</short_desc>
          <delta_ts>2024-01-15 17:21:35 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Forms</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Mac (Intel)</rep_platform>
          <op_sys>OS X 10.11</op_sys>
          <bug_status>ASSIGNED</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc>http://jsfiddle.net/FFXEc/</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>HasReduction, InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>159753</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Chris Rebert">webkit</reporter>
          <assigned_to name="Antonio Gomes">tonikitoo</assigned_to>
          <cc>ahmad.saleem792</cc>
    
    <cc>hnrch02+webkit</cc>
    
    <cc>jonlee</cc>
    
    <cc>ntim</cc>
    
    <cc>p.jacquemart</cc>
    
    <cc>roger_fong</cc>
    
    <cc>simon.fraser</cc>
    
    <cc>tonikitoo</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>zalan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1038345</commentid>
    <comment_count>0</comment_count>
    <who name="Chris Rebert">webkit</who>
    <bug_when>2014-09-30 14:24:29 -0700</bug_when>
    <thetext>OS Version: OS X 10.9.5
Safari Version: 7.1 (9537.85.10.17.1)

Steps to reproduce the problem:
1. Open the demo JS Fiddle ( http://jsfiddle.net/FFXEc/ ) in Safari.
2. (All further instructions are regarding the second/middle input[type=number] field on the demo.)
[Bug A]:
3. Hover the cursor up and down along the inner right edge of the field, in horizontal alignment with the increment and decrement buttons.
[Bug B/C]:
4. Click on the top part of the decrement button, such that you observe normal behavior.
5. Continue clicking repeatedly while moving your cursor downward, beyond the decrement button but still within the bounds of the input field.

What is the expected behavior?
A: The cursor ought to always remain a pointer while it&apos;s within the bounds of the decrement button.
B: Clicking outside the bounds of the increment/decrement buttons should not cause the input field&apos;s value to change.
C: Invoking the decrement button should always cause it to flash blue to indicate that it was clicked.

What went wrong?
A: The cursor changes from a pointer to an &quot;I&quot;-bar (for text entry/selection) before you have gone outside the bounds of the decrement button.
B: The field&apos;s number value continues to decrement.
C: The decrement button did not flash blue to indicate that it was somehow clicked.

Original Bootstrap bug report: https://github.com/twbs/bootstrap/issues/8350
Related Chromium bug: https://code.google.com/p/chromium/issues/detail?id=337668</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1044893</commentid>
    <comment_count>1</comment_count>
    <who name="Chris Rebert">webkit</who>
    <bug_when>2014-10-29 16:04:48 -0700</bug_when>
    <thetext>Still broken in Safari version 8.0 (10600.1.25) on OS X Yosemite.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1045246</commentid>
    <comment_count>2</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2014-10-31 00:03:12 -0700</bug_when>
    <thetext>&lt;rdar://problem/18834768&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1092254</commentid>
    <comment_count>3</comment_count>
      <attachid>252499</attachid>
    <who name="Chris Rebert">webkit</who>
    <bug_when>2015-05-06 11:44:29 -0700</bug_when>
    <thetext>Created attachment 252499
Copy of JS Fiddle example</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1162795</commentid>
    <comment_count>4</comment_count>
    <who name="Chris Rebert">webkit</who>
    <bug_when>2016-02-07 16:18:43 -0800</bug_when>
    <thetext>Still reproduces in Safari version 9.0.3 (11601.4.4)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1186668</commentid>
    <comment_count>5</comment_count>
    <who name="Antonio Gomes">tonikitoo</who>
    <bug_when>2016-04-23 15:18:55 -0700</bug_when>
    <thetext>(In reply to comment #4)
&gt; Still reproduces in Safari version 9.0.3 (11601.4.4)

I will take a look at this.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1186704</commentid>
    <comment_count>6</comment_count>
    <who name="alan">zalan</who>
    <bug_when>2016-04-23 19:41:17 -0700</bug_when>
    <thetext>It&apos;s probably because the placeholder renderer for the increment/decrement button is the height of the button element while the actual widget (for obvious reasons) is not scaled up to it. The fix is to size and position the placeholder renderer in sync with the  widget.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1186712</commentid>
    <comment_count>7</comment_count>
    <who name="alan">zalan</who>
    <bug_when>2016-04-23 21:25:18 -0700</bug_when>
    <thetext>(In reply to comment #6)
&gt; It&apos;s probably because the placeholder renderer for the increment/decrement
&gt; button is the height of the button element while the actual widget (for
&gt; obvious reasons) is not scaled up to it. The fix is to size and position the
&gt; placeholder renderer in sync with the  widget.
Though it might be a bit difficult as it&apos;s all flexbox driven afair.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1978136</commentid>
    <comment_count>8</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2023-09-18 15:55:16 -0700</bug_when>
    <thetext>Fixed recently in Chromium / Blink as well - https://chromium.googlesource.com/chromium/src.git/+/f75283abf000d956d99336f5762c4ff215ef44fb

Might not be exact fix but good to at least tag for even importing text etc.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2005121</commentid>
    <comment_count>9</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2024-01-15 10:50:31 -0800</bug_when>
    <thetext>(In reply to Ahmad Saleem from comment #8)
&gt; Fixed recently in Chromium / Blink as well -
&gt; https://chromium.googlesource.com/chromium/src.git/+/
&gt; f75283abf000d956d99336f5762c4ff215ef44fb
&gt; 
&gt; Might not be exact fix but good to at least tag for even importing text etc.

Just checked and we can merge this, it seems.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2005164</commentid>
    <comment_count>10</comment_count>
    <who name="Tim Nguyen (:ntim)">ntim</who>
    <bug_when>2024-01-15 16:34:41 -0800</bug_when>
    <thetext>(In reply to Ahmad Saleem from comment #9)
&gt; (In reply to Ahmad Saleem from comment #8)
&gt; &gt; Fixed recently in Chromium / Blink as well -
&gt; &gt; https://chromium.googlesource.com/chromium/src.git/+/
&gt; &gt; f75283abf000d956d99336f5762c4ff215ef44fb
&gt; &gt; 
&gt; &gt; Might not be exact fix but good to at least tag for even importing text etc.
&gt; 
&gt; Just checked and we can merge this, it seems.

Feel free to submit a PR :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2005170</commentid>
    <comment_count>11</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2024-01-15 17:03:24 -0800</bug_when>
    <thetext>(In reply to Tim Nguyen (:ntim) from comment #10)
&gt; (In reply to Ahmad Saleem from comment #9)
&gt; &gt; (In reply to Ahmad Saleem from comment #8)
&gt; &gt; &gt; Fixed recently in Chromium / Blink as well -
&gt; &gt; &gt; https://chromium.googlesource.com/chromium/src.git/+/
&gt; &gt; &gt; f75283abf000d956d99336f5762c4ff215ef44fb
&gt; &gt; &gt; 
&gt; &gt; &gt; Might not be exact fix but good to at least tag for even importing text etc.
&gt; &gt; 
&gt; &gt; Just checked and we can merge this, it seems.
&gt; 
&gt; Feel free to submit a PR :)

Doing local build but it might involve to merge this as well: https://source.chromium.org/chromium/chromium/src/+/2e5cf9f34b1ee2d0616be5f29501166021006416</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2005176</commentid>
    <comment_count>12</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2024-01-15 17:21:35 -0800</bug_when>
    <thetext>(In reply to Ahmad Saleem from comment #8)
&gt; Fixed recently in Chromium / Blink as well -
&gt; https://chromium.googlesource.com/chromium/src.git/+/
&gt; f75283abf000d956d99336f5762c4ff215ef44fb
&gt; 
&gt; Might not be exact fix but good to at least tag for even importing text etc.

Merging just this definitely leads to regression and in some cases the increment button get stuck and even if we lose focus, it continues to increment till we don&apos;t refresh page. Just noticed on local build using test case from Comment 0.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>252499</attachid>
            <date>2015-05-06 11:44:29 -0700</date>
            <delta_ts>2015-05-06 11:44:29 -0700</delta_ts>
            <desc>Copy of JS Fiddle example</desc>
            <filename>example-FFXEc.html</filename>
            <type>text/html</type>
            <size>1028</size>
            <attacher name="Chris Rebert">webkit</attacher>
            
              <data encoding="base64">PCFET0NUWVBFIGh0bWw+CjxodG1sPjxoZWFkPgogIDxtZXRhIGh0dHAtZXF1aXY9ImNvbnRlbnQt
dHlwZSIgY29udGVudD0idGV4dC9odG1sOyBjaGFyc2V0PVVURi04Ij4KICA8dGl0bGU+IC0ganNG
aWRkbGUgZGVtbzwvdGl0bGU+CiAgICAKICA8c3R5bGUgdHlwZT0idGV4dC9jc3MiPgogICAgLmN1
c3RvbS1mb250LXNpemUgewogICAgaGVpZ2h0OiAzNHB4OwogICAgCn0KaW5wdXRbdHlwZT0ibnVt
YmVyIl0gewogICAgZm9udC1zaXplOiAxNXB4Owp9CgojZml4ZWQ6Oi13ZWJraXQtb3V0ZXItc3Bp
bi1idXR0b24sCiNmaXhlZDo6LXdlYmtpdC1pbm5lci1zcGluLWJ1dHRvbiB7CiAgICBoZWlnaHQ6
IGF1dG87Cn0KCiAgPC9zdHlsZT4KCgo8L2hlYWQ+Cjxib2R5PgogIDxwPkEgbm9ybWFsIG51bWJl
ciBpbnB1dCBmb3IgY29tcGFyaXNvbjo8L3A+CjxpbnB1dCB0eXBlPSJudW1iZXIiIHZhbHVlPSI5
OSIgaWQ9ImNvbnRyb2wiPgoKPHA+VHJ5IGhvdmVyaW5nIGFuZCBjbGlja2luZyBvbiB0aGUgbG93
ZXIgaGFsZiBvZiB0aGUgZGVjcmVtZW50IGJ1dHRvbi4gTm90ZSBob3cgdGhlIGN1cnNvciBwcmVt
YXR1cmVseSBjaGFuZ2VzIHRvIGFuIEktYmFyIHRleHQgY3Vyc29yLiBZb3UgY2FuIGFsc28gbWFu
YWdlIHRvIGNsaWNrIGFuZCBhY3RpdmF0ZSB0aGUgZGVjcmVtZW50IGJ1dHRvbiBldmVuIHRob3Vn
aCB0aGUgY3Vyc29yIHNlZW1zIHRvIGJlIGJlbG93IHRoZSBidXR0b24gYW5kIHRoZSBidXR0b24g
ZG9lc24ndCBmbGFzaCBibHVlIHRvIGluZGljYXRlIHRoYXQgaXQgd2FzIGNsaWNrZWQuPC9wPgo8
aW5wdXQgdHlwZT0ibnVtYmVyIiB2YWx1ZT0iOTkiIGNsYXNzPSJjdXN0b20tZm9udC1zaXplIiBp
ZD0iYnJva2VuIj4KCjxicj48YnI+Cgo8cD5IZXJlJ3MgdGhlIENTUyBoYWNrL3dvcmthcm91bmQg
dGhhdCBzb21lIGZvbGtzIGFyZSB1c2luZzo8L3A+CjxpbnB1dCB0eXBlPSJudW1iZXIiIHZhbHVl
PSI0MiIgY2xhc3M9ImN1c3RvbS1mb250LXNpemUiIGlkPSJmaXhlZCI+Cgo8L2JvZHk+PC9odG1s
Pgo=
</data>

          </attachment>
      

    </bug>

</bugzilla>