93760 2012-08-10 21:08:08 -0700 Selection change caused by click event on button should be *after* the onclick fires 2022-07-26 22:50:13 -0700 1 1 1 Unclassified WebKit HTML Editing 528+ (Nightly build) Mac (Intel) OS X 10.8 NEW InRadar P2 Normal --- 1 dgatwood webkit-unassigned ahmad.saleem792 ap bfulgham rniwa tonikitoo webkit-bug-importer oldest_to_newest 693046 0 157856 dgatwood 2012-08-10 21:08:08 -0700 Created attachment 157856 Demonstrates odd behavior of buttons inside contentEditable regions I would normally expect a button's onclick handler to fire prior to any in-browser modification of the current DOM selection. For some odd reason, in WebKit, the reverse occurs—the selection changes first—leading to spectacular misbehavior if you're crazy enough to place a button inside a contentEditable region. :-) Test case attached. The expected behavior is that when you click inside the text above, then click the button, the click event should fire first, causing the content to be inserted at the selection point. Instead, the insertion point moves first, so that the insertion point is inside the text of the button. Thus, the content gets inserted inside the button. 693761 1 ap 2012-08-13 09:56:57 -0700 See also: bug 93674. Perhaps we match IE here, as well? 694122 2 rniwa 2012-08-13 14:53:45 -0700 IE doesn't change the selection until click event either. 1886921 3 ahmad.saleem792 2022-07-26 15:05:25 -0700 I am able to reproduce this bug using attached test case in Safari 15.6 on macOS 12.5 where placing caret on the text and then pressing button trigger dialog with "fail" rather than adding text. It is same with Chrome Canary 106 and only Firefox Nightly 104 is where it add text and does not show any "fail" dialog. Thanks! 1886969 4 webkit-bug-importer 2022-07-26 19:40:16 -0700 <rdar://problem/97641753> 1887015 5 rniwa 2022-07-26 22:50:13 -0700 Blink and WebKit still fail the attached test case. 157856 2012-08-10 21:08:08 -0700 2012-08-10 21:08:08 -0700 Demonstrates odd behavior of buttons inside contentEditable regions inserthtmlWrongPlace2.html text/html 613 dgatwood PGh0bWw+CjxoZWFkPgo8c2NyaXB0IHR5cGU9InRleHQvamF2YXNjcmlwdCI+PCEtLQoKZnVuY3Rp b24gdGVzdGl0KG91dGVyLCBpbm5lcikKewogICAgaWYgKCFkb2N1bWVudC5leGVjQ29tbWFuZCgi aW5zZXJ0SFRNTCIsIGZhbHNlLCAiPHNwYW4gaWQ9J3Rlc3RzcGFuJz5BZGRlZCB0ZXh0PC9zcGFu PiIpKSB7CglhbGVydCgiRXBpYyBmYWlsIik7CiAgICB9CgogICAgaWYgKGRvY3VtZW50LmdldEVs ZW1lbnRCeUlkKCJteWJ1dHRvbiIpLmlubmVySFRNTCA9PSAiQ2xpY2sgdG8gYWRkIHRleHQgaW4g dGhlIHRleHQgYWJvdmUuIikgewoJYWxlcnQoInBhc3MiKTsKICAgIH0gZWxzZSB7CglhbGVydCgi ZmFpbCIpOwogICAgfQp9CgovLy0tPjwvc2NyaXB0Pgo8L2hlYWQ+Cjxib2R5IGNvbnRlbnRlZGl0 YWJsZT0idHJ1ZSI+Cgo8ZGl2IGlkPSJzY3JhdGNocGFkIj5QdXQgdGhlIGN1cnNvciBzb21ld2hl cmUgaW4gdGhpcyB0ZXh0LCB0aGVuIGNsaWNrIHRoZSBidXR0b24gYmVsb3cuPC9kaXY+Cgo8YnV0 dG9uIGlkPSJteWJ1dHRvbiIgb25jbGljaz0idGVzdGl0KCk7Ij5DbGljayB0byBhZGQgdGV4dCBp biB0aGUgdGV4dCBhYm92ZS48L2J1dHRvbj4KPC9ib2R5Pgo8L2h0bWw+Cg==