12243 2007-01-13 02:55:05 -0800 FCKeditor: Paste buttons are always disabled 2008-01-30 06:57:16 -0800 1 1 1 Unclassified WebKit HTML Editing 420+ Mac OS X 10.4 RESOLVED DUPLICATE 11335 http://www.fckeditor.net/nightly/browsers_test.html P2 Normal --- 9915 1 webkit webkit-unassigned justin.garcia oldest_to_newest 35227 0 webkit 2007-01-13 02:55:05 -0800 The paste (Normal Paste, Paste as Plain Text and Paste from Word) buttons never get enabled in the toolbar or in the context menu. 10032 1 justin.garcia 2007-05-14 15:28:37 -0700 *** This bug has been marked as a duplicate of 11335 *** 7855 2 webkit 2007-06-12 04:19:33 -0700 (In reply to comment #1) > > *** This bug has been marked as a duplicate of 11335 *** > Justin, this bug seams to be related to queryCommandEnabled('Paste'), not to execCommand('Paste'). The 'Paste' command is one of commands defined in the execCommand "standards". So, it is extensively adopted. Here we have many things to think about. I'll try to give some justifications to make queryCommandEnabled work properly in this case, reflecting the Firefox behavior. Even if WebKit decides that the execCommand('Paste') will never paste user's data, for security reasons (even if the user wanted that), queryCommandEnabled('Paste') is quite useful for us to understand that there is data available in the clipboard. So, it would be ok if queryCommandEnabled('Paste') returns "true" and execCommand('Paste') returns "false". In FCKeditor, this is important so we can enable the interface buttons for pasting (using queryCommandEnabled). If we try to paste and execCommand returns false, we display a dialog to the end user, asking her to paste in that dialog by using the keyboard shortcuts. This is the current behavior with Firefox, and it would be wonderful to have it working in the same way with Safari. I'm adding a TC to it, which shows the Safari X Firefox difference. 7850 3 14977 webkit 2007-06-12 04:20:37 -0700 Created attachment 14977 TC 7648 4 justin.garcia 2007-06-13 14:20:23 -0700 I have a fix for this that Geoff has reviewed. I'll be checking it in soon. 62001 5 eric 2007-11-22 11:32:52 -0800 Just curious whatever happened to Justin's fix... did it get checked in? The test case still fails on TOT. 68758 6 justin.garcia 2008-01-29 19:58:54 -0800 This *is* a dupe of 11335 as far as I can tell. The test case outputs false for queryCommandEnabled('Paste') because it *is* disabled, because of the security concerns mentioned in 11335. Frederico, if I've got this wrong, please reopen. *** This bug has been marked as a duplicate of 11335 *** 68800 7 webkit 2008-01-30 06:57:16 -0800 In theory, the current behavior is correct. In practice, it is a problem for us, as there is now way for us to simply know that there is data available in the clipboard (see comment #2). So now it is up to you guys (even if I can predict the way you are going to take with it). 14977 2007-06-12 04:20:37 -0700 2007-06-12 04:20:37 -0700 TC paste.html text/html 562 webkit PCFET0NUWVBFIGh0bWwgUFVCTElDICItLy9XM0MvL0RURCBYSFRNTCAxLjAgVHJhbnNpdGlvbmFs Ly9FTiIgImh0dHA6Ly93d3cudzMub3JnL1RSL3hodG1sMS9EVEQveGh0bWwxLXRyYW5zaXRpb25h bC5kdGQiPg0KPGh0bWwgeG1sbnM9Imh0dHA6Ly93d3cudzMub3JnLzE5OTkveGh0bWwiPg0KPGhl YWQ+DQoJPHRpdGxlPlNhZmFyaSBCdWcgVEM8L3RpdGxlPg0KCTxzY3JpcHQgdHlwZT0idGV4dC9q YXZhc2NyaXB0Ij4NCg0Kd2luZG93Lm9ubG9hZCA9IGZ1bmN0aW9uKCkNCnsNCglkb2N1bWVudC5k ZXNpZ25Nb2RlID0gJ29uJyA7DQoJZG9jdW1lbnQuYm9keS5jb250ZW50RWRpdGFibGUgPSB0cnVl IDsNCg0KCWFsZXJ0KCBkb2N1bWVudC5xdWVyeUNvbW1hbmRFbmFibGVkKCAnUGFzdGUnICkgKSA7 DQp9DQoNCg0KCTwvc2NyaXB0Pg0KPC9oZWFkPg0KPGJvZHk+DQoJPHA+DQoJCUlmIHlvdSBoYXZl IGRhdGEgaW4gdGhlIGNsaXBib2FyZCwgYW4gYWxlcnQgd2l0aCAiPHN0cm9uZz50cnVlPC9zdHJv bmc+IiBzaG91bGQgYmUgZGlzcGxheWVkLjwvcD4NCjwvYm9keT4NCjwvaHRtbD4NCg==