Bug 198852

Summary:

[iOS] Selection handles "Lollipops" can become gray and stay gray

Product:

WebKit

Reporter:

Daniel Bates <dbates>

Component:

HTML Editing

Assignee:

Daniel Bates <dbates>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

graouts, jonlee, megan_gardner, webkit-bug-importer, wenson_hsieh

Priority:

Keywords:

InRadar, PlatformOnly

Version:

WebKit Local Build

Hardware:

iPhone / iPad

OS:

iOS 12

Bug Depends on:

Bug Blocks:

199685

Attachments:

Description	Flags
Screen capture - ignore my initial trouble getting the selection I wanted 😀	none
Patch	none

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.