Bug 198852 - [iOS] Selection handles "Lollipops" can become gray and stay gray
Summary: [iOS] Selection handles "Lollipops" can become gray and stay gray
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: WebKit Local Build
Hardware: iPhone / iPad iOS 12
: P2 Normal
Assignee: Daniel Bates
URL:
Keywords: InRadar, PlatformOnly
Depends on:
Blocks: 199685
  Show dependency treegraph
 
Reported: 2019-06-13 23:00 PDT by Daniel Bates
Modified: 2019-07-10 15:07 PDT (History)
5 users (show)

See Also:


Attachments
Screen capture - ignore my initial trouble getting the selection I wanted 😀 (5.91 MB, video/quicktime)
2019-06-13 23:00 PDT, Daniel Bates
no flags Details
Patch (1.72 KB, patch)
2019-07-06 15:09 PDT, Daniel Bates
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Bates 2019-06-13 23:00:55 PDT
Created attachment 372108 [details]
Screen capture - ignore my initial trouble getting the selection I wanted 😀

Steps to reproduce:

Ensure that your are signed into bugs.webkit.org and that you have Mail setup.

1. Visit <https://bugs.webkit.org/show_bug.cgi?id=198834>.
2. Select some text, say the word "test" in the description of that bug.
3. Tap my name "Daniel Bates" in the attachment listing to open a mail compose sheet.
4. Close the sheet.

Then the selection "lollipops" are gray. Moreover, you can interact with them, extend the selection, etc and they stay gray.

(BTW most of the time the report steps above work as-is. One time, I had to repeat steps 3 and 4 to get this bug to reproduce)
Comment 1 Radar WebKit Bug Importer 2019-06-13 23:02:37 PDT
<rdar://problem/51736130>
Comment 2 Daniel Bates 2019-07-06 15:09:54 PDT
Created attachment 373584 [details]
Patch
Comment 3 Jon Lee 2019-07-10 14:15:14 PDT
What's the next step here?
Comment 4 Daniel Bates 2019-07-10 14:58:06 PDT
(In reply to Jon Lee from comment #3)
> What's the next step here?

I was going to try to write an API test for this, but I already moved on to another bug and I probably won't circle back to add the test for a few days.
Comment 5 Daniel Bates 2019-07-10 15:04:32 PDT
Pinged Wenson on IRC and he is okay with landing the patch without a test for now.
Comment 6 Daniel Bates 2019-07-10 15:06:04 PDT
Filed bug #199685 to track adding a test.
Comment 7 Daniel Bates 2019-07-10 15:07:15 PDT
Comment on attachment 373584 [details]
Patch

Clearing flags on attachment: 373584

Committed r247328: <https://trac.webkit.org/changeset/247328>
Comment 8 Daniel Bates 2019-07-10 15:07:16 PDT
All reviewed patches have been landed.  Closing bug.