...
<rdar://problem/74415537>
Created attachment 420574 [details] WIP
Created attachment 420669 [details] Patch
Created attachment 420725 [details] Patch
Created attachment 420758 [details] Tests found using script
Created attachment 420795 [details] Patch
Created attachment 420865 [details] Patch
Created attachment 420922 [details] Patch
Comment on attachment 420922 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=420922&action=review > LayoutTests/fast/forms/ios/select-picker-change-and-focus-another-select.html:1 > +<!DOCTYPE html> <!-- webkit-test-runner [ useFlexibleViewport=true IOSFormControlRefreshEnabled=false ] --> While some of these tests are no longer relevant, we should file followup bugs to make some of these tests work using the new (refreshed) UI. This seems like one that might potentially still be applicable? (Maybe only with a hardware keyboard attached though) > LayoutTests/platform/ipad/fast/forms/choose-select-option.html:1 > +<!DOCTYPE html> <!-- webkit-test-runner [ useFlexibleViewport=true IOSFormControlRefreshEnabled=false ] --> This seems like another test that could (feasibly) be adapted to the refreshed select UI.
(In reply to Wenson Hsieh from comment #9) > Comment on attachment 420922 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=420922&action=review > > > LayoutTests/fast/forms/ios/select-picker-change-and-focus-another-select.html:1 > > +<!DOCTYPE html> <!-- webkit-test-runner [ useFlexibleViewport=true IOSFormControlRefreshEnabled=false ] --> > > While some of these tests are no longer relevant, we should file followup > bugs to make some of these tests work using the new (refreshed) UI. This > seems like one that might potentially still be applicable? (Maybe only with > a hardware keyboard attached though) This one was actually only relevant on iPhones, where it was possible to tap a select element while the picker was displayed for another element. It was never relevant on iPads since the tap would just dismiss the presented popover and an additional tap would be required to focus the other element. However, now that we present a modal view / context menu to choose options, it's no longer possible to immediately tap a select element while choosing options for another, since the first tap would dismiss the modal view / context menu. But I will take a look and see if there are other tests that can be adapted. > > LayoutTests/platform/ipad/fast/forms/choose-select-option.html:1 > > +<!DOCTYPE html> <!-- webkit-test-runner [ useFlexibleViewport=true IOSFormControlRefreshEnabled=false ] --> > > This seems like another test that could (feasibly) be adapted to the > refreshed select UI. This test has already been adapted to the new UI – see LayoutTests/fast/forms/ios/form-control-refresh/select/choose-select-option.html.
Committed r273154: <https://commits.webkit.org/r273154> All reviewed patches have been landed. Closing bug and clearing flags on attachment 420922 [details].