26443 2009-06-16 06:57:55 -0700 [:first-letter][createlink][contenteditable] Safari creates a link on wrong characters with document.execCommand("createlink", false, "somelink") if the contenteditable element has a :first-letter rule applied to it. 2022-06-23 22:08:22 -0700 1 1 1 Unclassified WebKit HTML Editing 528+ (Nightly build) Mac (Intel) OS X 10.5 RESOLVED CONFIGURATION CHANGED P1 Normal --- 6185 58911 1 dpopa webkit-unassigned ahmad.saleem792 ap enrica rniwa thaddee6yann tkent oldest_to_newest 126196 0 dpopa 2009-06-16 06:57:55 -0700 Safari crashes on document.execCommand("createlink", false, "somelink") if the contenteditable element has a :first-letter rule applied to it. Steps to reproduce: 1. have the following in a page: <style> .someclass:first-letter { font-weight: bold; } </style> <div class="someclass" contenteditable="true">contentEditable element. Some text and some words.</div> <input type="button" value="Create link" onclick="document.execCommand('createlink', false, 'http://www.google.com')" /> 2. Select something from the contenteditable DIV. 3. Click the button. Result: Safari crashes after a few seconds. A few details, notes on this behavior: Note 1: Can't select the first character/letter, see bug #6185 Note 2: Safari doesn't crash if the selection is inside the first word and it starts before or after the second character in the first word. The selection is shifted one character to the left. The contenteditable element cannot be focused anymore after createlink if the selection started before the second character, only the link can be focused in this case. See attachment for testcase. 126197 1 31348 dpopa 2009-06-16 06:59:43 -0700 Created attachment 31348 Testcase showing described behavior 126198 2 dpopa 2009-06-16 07:01:22 -0700 Possible related bugs: bug #22256 Reproduceable crash with @font-face and :first-letter pseudo class bug #15602 Quirksmode: CSS1: WebKit fails dynamic :first-letter test bug #14550 Non-layout style change does not update nested first-letter bug #6185 Bug 6185: CSS1: character styled with :first-letter is not selectable 126199 3 dpopa 2009-06-16 07:08:35 -0700 No crash in Safari 3.2.3 But the selection is shifted one character to the left and clicking after the newly created link doesn't set the selection at the click location (same as in the second note from the bug description) 175205 4 thaddee6yann 2009-12-28 02:32:11 -0800 Possibly related issues: * any text insertion is shifted by a seemingly arbitrary number of characters, usually 1. * a character entered at the end duplicates the line and sometimes deletes a number of characters at the end of the line. Note: this is only true on the first line. 262023 5 tkent 2010-08-09 01:51:48 -0700 I took a look at the code, and my conclusion is we should close Bug#6185 before this. ("close" means to fix it or to decide we won't fix it.) 1877963 6 ahmad.saleem792 2022-06-23 15:58:21 -0700 Based on attached test and following steps from Description, I am not able to crash Safari 15.5 on macOS 12.4. This was fixed long back as commented in Comment 03. Although - first letter non-selection bug is still present on attached test case (for which we have separate bug - https://bugs.webkit.org/show_bug.cgi?id=6185). Although, Comment 05 mentions that this bug should remain open till some conclusion is reached for the bug). IMO - We can close this since it is not crashing and leave 6185 open for fixing since now all other browsers have fixed it and first letter is selectable. Thanks! 1878115 7 rniwa 2022-06-23 22:08:22 -0700 Safari doesn't hit any crash anymore. 31348 2009-06-16 06:59:43 -0700 2009-06-16 06:59:43 -0700 Testcase showing described behavior first-letter-createlink-crash.html text/html 1409 dpopa PCFET0NUWVBFIGh0bWwgUFVCTElDICItLy9XM0MvL0RURCBYSFRNTCAxLjAgVHJhbnNpdGlvbmFs Ly9FTiINCgkiaHR0cDovL3d3dy53My5vcmcvVFIveGh0bWwxL0RURC94aHRtbDEtdHJhbnNpdGlv bmFsLmR0ZCI+DQo8aHRtbD4NCjxoZWFkPg0KPHRpdGxlPjpmaXJzdC1sZXR0ZXIgY3JlYXRlbGlu ayBjb250ZW50ZWRpdGFibGUgY3Jhc2g8L3RpdGxlPg0KPHN0eWxlPg0KLnNvbWVjbGFzczpmaXJz dC1sZXR0ZXIgew0KCWZvbnQtd2VpZ2h0OiBib2xkOw0KfQ0KPC9zdHlsZT4NCjwvaGVhZD4NCg0K PGJvZHk+DQo8aDM+OmZpcnN0LWxldHRlciBjcmVhdGVsaW5rIGNvbnRlbnRlZGl0YWJsZSBidWc6 IDwvaDM+DQo8b2w+DQoJPGxpPlNlbGVjdCBzb21ldGhpbmcgZnJvbSB0aGUgY29udGVudGVkaXRh YmxlIERJViBiZWxvdzwvbGk+DQoJPGxpPkNsaWNrIHRoZSAiQ3JlYXRlIGxpbmsiIGJ1dHRvbiB0 byBpbnZva2UgdGhlIGRvY3VtZW50LmV4ZWNDb21tYW5kKCJjcmVhdGVsaW5rIiwgZmFsc2UsICJo dHRwOi8vd3d3Lmdvb2dsZS5jb20iKSBjb21tYW5kPC9saT4NCjwvb2w+DQo8ZGl2Pg0KCVJlc3Vs dDogU2FmYXJpIGNyYXNoZXMgYWZ0ZXIgYSBmZXcgc2Vjb25kcy4NCjwvZGl2Pg0KPGJyIC8+DQo8 aW5wdXQgdHlwZT0iYnV0dG9uIiB2YWx1ZT0iQ3JlYXRlIGxpbmsiIG9uY2xpY2s9ImRvY3VtZW50 LmV4ZWNDb21tYW5kKCdjcmVhdGVsaW5rJywgZmFsc2UsICdodHRwOi8vd3d3Lmdvb2dsZS5jb20n KSIgLz4NCjxiciAvPg0KPGJyIC8+DQo8ZGl2IGNsYXNzPSJzb21lY2xhc3MiIGNvbnRlbnRlZGl0 YWJsZT0idHJ1ZSI+Y29udGVudEVkaXRhYmxlIGVsZW1lbnQuIFNvbWUgdGV4dCBhbmQgc29tZSB3 b3Jkcy48L2Rpdj4NCjxiciAvPg0KPGRpdj4NCglOb3RlIDE6IENhbid0IHNlbGVjdCB0aGUgZmly c3QgY2hhcmFjdGVyL2xldHRlciwgc2VlIGJ1ZyA8YSBocmVmPSJodHRwczovL2J1Z3Mud2Via2l0 Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NjE4NSI+NjE4NTwvYT48YnIgLz4NCglOb3RlIDI6IFNhZmFy aSBkb2Vzbid0IGNyYXNoIGlmIHRoZSBzZWxlY3Rpb24gaXMgaW5zaWRlIHRoZSBmaXJzdCB3b3Jk IGFuZCBpdCBzdGFydHMgYmVmb3JlIG9yIGFmdGVyIHRoZSBzZWNvbmQgY2hhcmFjdGVyIGluIHRo ZSBmaXJzdCB3b3JkLiBUaGUgc2VsZWN0aW9uIGlzIHNoaWZ0ZWQgb25lIGNoYXJhY3RlciB0byB0 aGUgbGVmdC48YnIgLz4NCglUaGUgY29udGVudGVkaXRhYmxlIGVsZW1lbnQgY2Fubm90IGJlIGZv Y3VzZWQgYW55bW9yZSBhZnRlciBjcmVhdGVsaW5rIGlmIHRoZSBzZWxlY3Rpb24gc3RhcnRlZCBi ZWZvcmUgdGhlIHNlY29uZCBjaGFyYWN0ZXIsIG9ubHkgdGhlIGxpbmsgY2FuIGJlIGZvY3VzZWQg aW4gdGhpcyBjYXNlLg0KPC9kaXY+DQoNCjwvYm9keT4NCjwvaHRtbD4=