Don't copy if there's nothing to copy

5adf0808 · Douwe Maan · bc3448fc · 5adf0808
Commit 5adf0808 authored Jan 16, 2017 by Douwe Maan
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 0 deletions

app/assets/javascripts/copy_as_gfm.js.es6 app/assets/javascripts/copy_as_gfm.js.es6 +2 -0

No files found.
--- a/app/assets/javascripts/copy_as_gfm.js.es6
+++ b/app/assets/javascripts/copy_as_gfm.js.es6
@@ -239,6 +239,8 @@
      let selectedDocument = selection.getRangeAt(0).cloneContents();
      if (!selectedDocument) return;

+      if (selectedDocument.textContent.length === 0) return;
+
      e.preventDefault();
      clipboardData.setData('text/plain', selectedDocument.textContent);