clarify safari browser support

57fb038a · Mike Greiling · 2e5ce7e6 · 57fb038a
Commit 57fb038a authored Nov 28, 2017 by Mike Greiling
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 4 deletions

app/assets/javascripts/behaviors/copy_to_clipboard.js app/assets/javascripts/behaviors/copy_to_clipboard.js +4 -4

No files found.
--- a/app/assets/javascripts/behaviors/copy_to_clipboard.js
+++ b/app/assets/javascripts/behaviors/copy_to_clipboard.js
@@ -21,10 +21,10 @@ function genericSuccess(e) {
  $(e.trigger).blur();
 }
-// Safari doesn't support `execCommand`, so instead we inform the user to
+/**
-// copy manually.
+ * Safari > 10 doesn't support `execCommand`, so instead we inform the user to copy manually.
-//
+ * See http://clipboardjs.com/#browser-support
-// See http://clipboardjs.com/#browser-support
+ */
 function genericError(e) {
  let key;
  if (/Mac/i.test(navigator.userAgent)) {