RESOLVED FIXED 108428
Click on a label element won't select input[type=date]
https://bugs.webkit.org/show_bug.cgi?id=108428
Summary Click on a label element won't select input[type=date]
Kent Tamura
Reported 2013-01-30 22:27:59 PST
INPUT_MULTIPLE_FIELDS_UI: Click on a label element won't select input[type=date] https://code.google.com/p/chromium/issues/detail?id=172850 <label>Date: <input type=date></label> Click on the "Date:" label should focus on the date input. If it was type=test, the click would focus on it.
Attachments
Patch (11.86 KB, patch)
2013-01-30 22:45 PST, Kent Tamura
no flags
Patch 2 (11.97 KB, patch)
2013-01-31 00:19 PST, Kent Tamura
no flags
Patch 3 (12.90 KB, patch)
2013-01-31 01:14 PST, Kent Tamura
no flags
Patch 4 (12.31 KB, patch)
2013-01-31 01:16 PST, Kent Tamura
no flags
Kent Tamura
Comment 1 2013-01-30 22:45:44 PST
Kent Tamura
Comment 2 2013-01-31 00:19:52 PST
Kentaro Hara
Comment 3 2013-01-31 00:49:49 PST
Comment on attachment 185700 [details] Patch 2 View in context: https://bugs.webkit.org/attachment.cgi?id=185700&action=review > LayoutTests/fast/forms/date/date-click-on-label.html:2 > +<body id=body> Nit: id=body is not necessary. > LayoutTests/fast/forms/date/date-click-on-label.html:7 > +document.body.offsetLeft; Nit: What is it for? > LayoutTests/fast/forms/date/date-click-on-label.html:11 > +shouldBeEqualToString('document.activeElement.id', 'date1'); Shall we also check the value of document.activeElement.id before dispatching the click event?
Kent Tamura
Comment 4 2013-01-31 01:14:43 PST
Created attachment 185711 [details] Patch 3 Follow review comments
Kent Tamura
Comment 5 2013-01-31 01:16:33 PST
Created attachment 185712 [details] Patch 4 remove unrelated change
Kentaro Hara
Comment 6 2013-01-31 01:19:30 PST
Comment on attachment 185712 [details] Patch 4 Looks better.
WebKit Review Bot
Comment 7 2013-01-31 03:19:12 PST
Comment on attachment 185712 [details] Patch 4 Clearing flags on attachment: 185712 Committed r141395: <http://trac.webkit.org/changeset/141395>
WebKit Review Bot
Comment 8 2013-01-31 03:19:16 PST
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.