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
188035
Manually activate the selection assistant for Tap and a Half gestures
https://bugs.webkit.org/show_bug.cgi?id=188035
Summary
Manually activate the selection assistant for Tap and a Half gestures
Megan Gardner
Reported
2018-07-25 21:01:50 PDT
Manually activate the selection assistant for Tap and a Half gestures
Attachments
Patch
(2.78 KB, patch)
2018-07-25 21:04 PDT
,
Megan Gardner
thorton
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Megan Gardner
Comment 1
2018-07-25 21:04:53 PDT
Created
attachment 345816
[details]
Patch
Megan Gardner
Comment 2
2018-07-25 21:06:17 PDT
<
rdar://problem/42595472
>
Tim Horton
Comment 3
2018-07-25 21:39:27 PDT
Comment on
attachment 345816
[details]
Patch Eww? Also, test?
Simon Fraser (smfr)
Comment 4
2018-07-25 21:48:29 PDT
Comment on
attachment 345816
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=345816&action=review
> Source/WebKit/ChangeLog:11 > + is starting. This does not interfear with activation for other gestures, and it ensures that
interfere
> Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm:2731 > + // <
rdar://problem/42610532
> Find a better way to make sure that the selection assistant is activated for tap and a half gestures
Maybe don't include the radar title.
Tim Horton
Comment 5
2018-07-25 21:57:52 PDT
Comment on
attachment 345816
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=345816&action=review
>> Source/WebKit/ChangeLog:11 >> + is starting. This does not interfear with activation for other gestures, and it ensures that > > interfere
You don’t fear it?
>> Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm:2731 >> + // <
rdar://problem/42610532
> Find a better way to make sure that the selection assistant is activated for tap and a half gestures > > Maybe don't include the radar title.
Yeah, this can be one line even, with the radar in parens after fixme.
Megan Gardner
Comment 6
2018-07-26 12:05:20 PDT
Decided to go with a change in UIKit instead, as it was less hacky.
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