RESOLVED FIXED 38381
[Windows] Implement <input type=number> UI 
https://bugs.webkit.org/show_bug.cgi?id=38381
Summary [Windows] Implement <input type=number> UI
Kent Tamura
Reported 2010-04-30 02:01:45 PDT
Visit https://bugs.webkit.org/show_bug.cgi?id=27968 for a screenshot.
Attachments
Patch (9.91 KB, patch)
2010-07-11 08:09 PDT, Kent Tamura 		no flags
DetailsFormatted DiffDiff
Patch 2 (9.82 KB, patch)
2010-09-30 00:18 PDT, Kent Tamura 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Kent Tamura
Comment 1 2010-07-11 08:09:53 PDT
Created attachment 61177 [details] Patch
chris fleizach
Comment 2 2010-09-22 00:30:12 PDT
Comment on attachment 61177 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=61177&action=review mostly minor comments > WebCore/rendering/RenderThemeWin.cpp:98 > +// Spin button staes states spelled incorrectly > WebCore/rendering/RenderThemeWin.cpp:500 > +} extra new line that's not needed > WebCore/rendering/RenderThemeWin.cpp:516 > + result.m_state = determineClassicState(o, false); instead of using a bool here, a simple ENUM would be helpful so that when looking at lines like this, it's clear that you're passing in a down button > WebCore/rendering/RenderThemeWin.cpp:680 > + width = 17; // Vista's default. can you retrieve this from some system setting if it's a default? > WebCore/rendering/RenderThemeWin.cpp:688 > + return false; why less than 2 and not 1 or 3?
Kent Tamura
Comment 3 2010-09-30 00:18:11 PDT
Created attachment 69317 [details] Patch 2
Kent Tamura
Comment 4 2010-09-30 00:20:19 PDT
Comment on attachment 61177 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=61177&action=review >> WebCore/rendering/RenderThemeWin.cpp:98 >> +// Spin button staes > > states spelled incorrectly Fixed. >> WebCore/rendering/RenderThemeWin.cpp:500 >> +} > > extra new line that's not needed Fixed. >> WebCore/rendering/RenderThemeWin.cpp:516 >> + result.m_state = determineClassicState(o, false); > > instead of using a bool here, a simple ENUM would be helpful so that when looking at lines like this, it's clear that you're passing in a down button Fixed. I introduced ControlSubPart enum, which is the same as RenderThemeChromiumWin. >> WebCore/rendering/RenderThemeWin.cpp:680 >> + width = 17; // Vista's default. > > can you retrieve this from some system setting if it's a default? I don't think we can. >> WebCore/rendering/RenderThemeWin.cpp:688 >> + return false; > > why less than 2 and not 1 or 3? I added a comment.
chris fleizach
Comment 5 2010-09-30 09:13:09 PDT
Comment on attachment 69317 [details] Patch 2 r=me thanks
Kent Tamura
Comment 6 2010-09-30 22:09:19 PDT
Comment on attachment 69317 [details] Patch 2 Clearing flags on attachment: 69317 Committed r68864: <http://trac.webkit.org/changeset/68864>
Kent Tamura
Comment 7 2010-09-30 22:09:28 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.