18693 2008-04-23 00:17:38 -0700 Cmd-B and Cmd-I events cannot be overriden by JavaScript in editor mode. 2011-09-07 13:21:34 -0700 1 1 1 Unclassified WebKit HTML Editing 528+ (Nightly build) Mac OS X 10.5 NEW P2 Normal --- 1 martin webkit-unassigned ap ayg davidr dpopa eric mitz oldest_to_newest 78625 0 martin 2008-04-23 00:17:38 -0700 When editor mode is enabled, it is impossible to receive the keydown events of Meta-B and Meta-I in JavaScript. This is a serious problem for implementing web based editors because it is often desirable to implement custom logic for the hotkeys instead of always following the browser's default. To reproduce the issue: 1. Open the attached HTML test case. 2. Click in the iframe area, a caret should appear indicating the area is editable. 3. Press any Meta combination that's not Meta-I or Meta-B, the correct keycode should be printed out in the editing area. 4. Press Meta-I or Meta-B, the keycode is not printed out indicating the JavaScript logic failed to receive the events. 78626 1 20765 martin 2008-04-23 00:18:19 -0700 Created attachment 20765 Test case 78627 2 martin 2008-04-23 00:18:56 -0700 This issue is causing the bug https://dev.fckeditor.net/ticket/1701 in FCKeditor. 79148 3 ap 2008-04-28 10:53:16 -0700 Confirmed with r32635. IE allows JavaScript to cancel default processing for these events. 79156 4 mitz 2008-04-28 11:25:45 -0700 Similarly perhaps, Command-I cannot be used in Safari to invoke the Mail Contents of This Page command when in editable text (<rdar://problem/5845299>). 120424 5 ap 2009-05-08 08:50:43 -0700 *** Bug 25604 has been marked as a duplicate of this bug. *** 120477 6 eric 2009-05-08 15:41:28 -0700 I looked into this a little yesterday. We're short-circuiting handling of these events. I didn't quite know how to hook into the NSResponder/NSTextField model in order to process the key events at the right time however. It seems we might have to implement our own processKeyEvents: which is called from the EditorClient handleKeyEvent callback. I should post actual stack traces in the bug for more intelligent discussion. 20765 2008-04-23 00:18:19 -0700 2008-04-23 00:18:19 -0700 Test case 1701.html text/html 1522 martin PGh0bWw+Cgk8aGVhZD4KCQk8dGl0bGU+TWV0YSBob3RrZXlzIGNhbm5vdCBiZSBjYXB0dXJlZCBv biBGaXJlZm94IE1hYzwvdGl0bGU+CgkJPHNjcmlwdCB0eXBlPSJ0ZXh0L2phdmFzY3JpcHQiPgp3 aW5kb3cub25sb2FkID0gZnVuY3Rpb24oKQp7Cgl2YXIgZiA9IGRvY3VtZW50LmJvZHkuYXBwZW5k Q2hpbGQoZG9jdW1lbnQuY3JlYXRlRWxlbWVudCgnaWZyYW1lJykpOwoJZi5oZWlnaHQgPSA3MDA7 CglmLndpZHRoID0gOTAwOwoJZi5vbmxvYWQgPSBmdW5jdGlvbigpCgl7CgkJdmFyIGRvYyA9IGYu Y29udGVudFdpbmRvdy5kb2N1bWVudDsKCQlkb2MuYm9keS5pbm5lckhUTUwgPSAnPHA+Tm93IHRo ZXNlIHBvaW50cyBvZiBkYXRhIG1ha2UgYSBiZWF1dGlmdWwgbGluZS48YnI+JwoJCQkrICdBbmQg d2VcJ3JlIG91dCBvZiBiZXRhLjxicj4nCgkJCSsgJ1dlXCdyZSByZWxlYXNpbmcgb24gdGltZS48 YnI+JwoJCQkrICc8L3A+PHA+U28gSVwnbSBHTGFELiBJIGdvdCBidXJuZWQuPGJyPicKCQkJKyAn VGhpbmsgb2YgYWxsIHRoZSB0aGluZ3Mgd2UgbGVhcm5lZDxicj4nCgkJCSsgJ2ZvciB0aGUgcGVv cGxlIHdobyBhcmUgc3RpbGwgYWxpdmUuPC9wPicKCQlkb2MuZGVzaWduTW9kZSA9ICdvbic7CgkJ ZG9jLmFkZEV2ZW50TGlzdGVuZXIoJ2tleWRvd24nLCBmdW5jdGlvbihldnQpCgkJCQl7CgkJCQkJ ZG9jLmV4ZWNDb21tYW5kKCdpbnNlcnRodG1sJywgZmFsc2UsICI8ZGl2PktleUNvZGUgPSAiICsg ZXZ0LmtleUNvZGUgKyAiLCBDdHJsL01ldGEgPSAiICsgCgkJCQkJCQkoZXZ0LmN0cmxLZXkgfHwg ZXZ0Lm1ldGFLZXkpICsgIjwvZGl2PiIpOwoJCQkJCWV2dC5wcmV2ZW50RGVmYXVsdCgpOwoJCQkJ fSwgZmFsc2UpOwoJfQoJZi5zcmMgPSAnYWJvdXQ6YmxhbmsnOwp9CgkJPC9zY3JpcHQ+Cgk8L2hl YWQ+Cgk8Ym9keT4KCQk8cD5UaGlzIHRlc3QgY2FzZSBpcyBmb3IgR2Vja28gYW5kIFNhZmFyaSBv bmx5LjwvcD4KCQk8cD5GaXJlZm94OiBUcnkgcHJlc3NpbmcgdGhlIGRvY3VtZW50IG11dGF0aW5n IEN0cmwgb3IgTWV0YSBob3RrZXkgY29tYm9zIGluc2lkZSB0aGUgZWRpdGluZyBhcmVhIC0gaXQg d2lsbCBiZSBpbnRlcmNlcHRlZCBvbiBGaXJlZm94IGZvciBXaW5kb3dzIGJ1dCBub3QgRmlyZWZv eCBmb3IgTWFjLjwvcD4KCQk8cD5TYWZhcmkgTWFjOiBUcnkgcHJlc3NpbmcgTWV0YS1CIG9yIE1l dGEtSSBpbiB0aGUgZWRpdGluZyBhcmVhIC0gaXQgY2Fubm90IGJlIGludGVyY2VwdGVkIGJ5IEph dmFTY3JpcHQuPC9wPgoJCTxwPlNvbWUgY29tYm8ga2V5cyB0byB0cnk6PC9wPgoJCTx1bD4KCQkJ PGxpPkN1dCAtIEN0cmwvTWV0YS1YPC9saT4KCQkJPGxpPlBhc3RlIC0gQ3RybC9NZXRhLVY8L2xp PgoJCQk8bGk+VW5kbyAtIEN0cmwvTWV0YS1aPC9saT4KCQkJPGxpPlJlZG8gLSBDdHJsL01ldGEt WTwvbGk+CgkJCTxsaT5Cb2xkIC0gQ3RybC9NZXRhLUI8L2xpPgoJCQk8bGk+SXRhbGljcyAtIEN0 cmwvTWV0YS1JPC9saT4KCQk8L3VsPgoJPC9ib2R5Pgo8L2h0bWw+Cg==