WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED WONTFIX
195826
[iOS] Modalify <select> popover; app key commands should not be recognized when popover is open
https://bugs.webkit.org/show_bug.cgi?id=195826
Summary
[iOS] Modalify <select> popover; app key commands should not be recognized wh...
Daniel Bates
Reported
2019-03-15 15:23:14 PDT
Seen on iPad. Steps to reproduce: 1. Visit <data:text/html,<select><option>First</option><option>Second</option></select>>. 2. Tap the popup button to open the popover. 3. Press Command + r. Safari reloads the page! But that is not intuitive and does not match the Mac. Instead the popover should behave like a modal and all key events should go it or we need to make the illusions of this.
Attachments
Patch
(9.23 KB, patch)
2019-03-15 16:02 PDT
,
Daniel Bates
no flags
Details
Formatted Diff
Diff
Patch
(9.17 KB, patch)
2019-03-15 16:06 PDT
,
Daniel Bates
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2019-03-15 15:38:00 PDT
<
rdar://problem/48941551
>
Daniel Bates
Comment 2
2019-03-15 16:02:57 PDT
Created
attachment 364871
[details]
Patch
Daniel Bates
Comment 3
2019-03-15 16:06:36 PDT
Created
attachment 364874
[details]
Patch
Brent Fulgham
Comment 4
2019-03-18 21:23:07 PDT
Comment on
attachment 364874
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=364874&action=review
Did we still want to do this? From our earlier discussion I thought modal select was not the desired UI?
> Source/WebKit/ChangeLog:8 > + Fake it til you make it ð Add a hack to give the illusion that popovers are modal and all
Did you mean to include that non-ASCII character in this line?
Daniel Bates
Comment 5
2019-03-19 11:26:52 PDT
Comment on
attachment 364874
[details]
Patch No need to fake what you won't make.😀
Brent Fulgham
Comment 6
2019-03-19 12:43:17 PDT
We decided not to modality <select>.
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