Bug 182012

Summary: Emoji with skin tone modifier transformed into color swatch when trails selection in Page Source view
Product: WebKit Reporter: Daniel Bates <dbates>
Component: TextAssignee: Joseph Pecoraro <joepeck>
Status: NEW    
Severity: Normal CC: joepeck, mitz, mmaxfield, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Local Build   
Hardware: Mac   
OS: macOS 10.13   
Attachments:
Description Flags
Test
none
Screen capture
none
Screenshot
none
CodeMirror DOM Screenshot none

Daniel Bates
Reported 2018-01-23 14:12:24 PST
Created attachment 332070 [details] Test Open the attached test case in Safari and follow the steps.
Attachments
Test (514 bytes, text/html)
2018-01-23 14:12 PST, Daniel Bates 		no flags
Details
Screen capture (2.27 MB, video/quicktime)
2018-01-23 14:13 PST, Daniel Bates 		no flags
Details
Screenshot (213.11 KB, image/png)
2018-01-23 14:14 PST, Daniel Bates 		no flags
Details
CodeMirror DOM Screenshot (343.72 KB, image/png)
2018-01-23 15:12 PST, Joseph Pecoraro 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Daniel Bates
Comment 1 2018-01-23 14:13:35 PST
Created attachment 332071 [details] Screen capture Screen capture of the bug reproduced by using the attached test case.
Daniel Bates
Comment 2 2018-01-23 14:14:33 PST
Created attachment 332072 [details] Screenshot
Radar WebKit Bug Importer
Comment 3 2018-01-23 14:16:43 PST
<rdar://problem/36793439>
Daniel Bates
Comment 4 2018-01-23 14:16:48 PST
For some reason this only happens in the Page Source view. That is, selecting the faces in the main browser windows does not cause the next face to turn into a color swatch.
Joseph Pecoraro
Comment 5 2018-01-23 15:12:42 PST
Created attachment 332082 [details] CodeMirror DOM Screenshot I think this is an issue with CodeMirror's selection. CodeMirror is what draws the selection in the text editor (view source) view, and it has its own element for the Emoji without Color, then the following element contains the trailing color.
Joseph Pecoraro
Comment 6 2018-01-23 15:16:01 PST
We can test against CodeMirror trunk, if it reproduces there we can file a bug on CodeMirror. It sounds like CodeMirror's selection needs to take into account entire grapheme clusters.
Note You need to log in before you can comment on or make changes to this bug.