Cheeky comment on library about clipboard.js HOT 2 CLOSED

Hey Matt,

That's true. In fact, queryCommandSupported is pretty inconsistent.

Unfortunately, in versions of Firefox prior to 41, we returned true from document.queryCommandSupported("copy") even though the web page was unable to actually perform the copy operation. However, attempting to execute document.execCommand("copy") would throw a SecurityException. So, attempting to copy on load, and checking for this exception is probably the easiest way to feature-detect support for document.execCommand("copy") in Firefox.

Here's what @mystor suggested on his post:

var supported = document.queryCommandSupported("copy");
if (supported) {
  // Check that the browser isn't Firefox pre-41
  try {
    document.execCommand("copy");
  } catch (e) {
    supported = false;
  }
}
if (!supported) {
  // Fall back to an alternate approach like ZeroClipboard
}

I guess the best solution is to just ignore the queryCommandSupported and check what has returned from document.execCommand just like @eduardolundgren did ebc171b.

from clipboard.js.

Just updated the branch with fix for Safari.

from clipboard.js.