WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
98478
Fix font for date/time input types.
https://bugs.webkit.org/show_bug.cgi?id=98478
Summary
Fix font for date/time input types.
Kent Tamura
Reported
2012-10-04 21:46:46 PDT
* input[type=date] should use monospace font * input[type=datetime/datetime-local/time/week/month] should not use monospace font if the type is disabled.
Attachments
Patch
(2.47 KB, patch)
2012-10-04 22:01 PDT
,
Kent Tamura
no flags
Details
Formatted Diff
Diff
Patch 2
(2.60 KB, patch)
2012-10-04 22:05 PDT
,
Kent Tamura
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Kent Tamura
Comment 1
2012-10-04 22:01:27 PDT
Created
attachment 167245
[details]
Patch
Kent Tamura
Comment 2
2012-10-04 22:05:52 PDT
Created
attachment 167246
[details]
Patch 2 Update ChangeLog
Kentaro Hara
Comment 3
2012-10-04 22:07:41 PDT
Comment on
attachment 167246
[details]
Patch 2 View in context:
https://bugs.webkit.org/attachment.cgi?id=167246&action=review
> Source/WebCore/ChangeLog:17 > + The input[type=date] change is covered by > + fast/forms/date/calendar-picker-appearance.html, > + platform/chromium/fast/forms/date/date-suggestion-picker-appearance*.html
Why isn't there any change in these test results although you're changing fonts in this patch? Such a minor change is not detected?
Kent Tamura
Comment 4
2012-10-04 22:10:02 PDT
Comment on
attachment 167246
[details]
Patch 2 View in context:
https://bugs.webkit.org/attachment.cgi?id=167246&action=review
>> Source/WebCore/ChangeLog:17 >> + platform/chromium/fast/forms/date/date-suggestion-picker-appearance*.html > > Why isn't there any change in these test results although you're changing fonts in this patch? Such a minor change is not detected?
Fortunately these tests are suppressed in TestExpectations for now because they need rebaseline by prior changes.
Kentaro Hara
Comment 5
2012-10-04 22:10:49 PDT
Comment on
attachment 167246
[details]
Patch 2 OK. Please just confirm the appearance manually.
WebKit Review Bot
Comment 6
2012-10-04 23:24:48 PDT
Comment on
attachment 167246
[details]
Patch 2 Clearing flags on attachment: 167246 Committed
r130467
: <
http://trac.webkit.org/changeset/130467
>
WebKit Review Bot
Comment 7
2012-10-04 23:24:51 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.
Top of Page
Format For Printing
XML
Clone This Bug