From 0b1db60d8bb05a40a652aa9051df7768ad2322d9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Gina=20H=C3=A4u=C3=9Fge?= <gina@octoprint.org>
Date: Mon, 13 Nov 2017 14:10:54 +0100
Subject: [PATCH] Easier copying of terminal contents

---
 src/octoprint/static/js/app/helpers.js             |  2 +-
 src/octoprint/static/js/app/viewmodels/terminal.js |  4 ++++
 src/octoprint/templates/tabs/terminal.jinja2       | 14 +++++++++++---
 3 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/src/octoprint/static/js/app/helpers.js b/src/octoprint/static/js/app/helpers.js
index d5ab5d29..471a1edd 100644
--- a/src/octoprint/static/js/app/helpers.js
+++ b/src/octoprint/static/js/app/helpers.js
@@ -1154,7 +1154,7 @@ var escapeUnprintableCharacters = function(str) {
 };
 
 var copyToClipboard = function(text) {
-    var temp = $("<input>");
+    var temp = $("<textarea>");
     $("body").append(temp);
     temp.val(text).select();
     document.execCommand("copy");
diff --git a/src/octoprint/static/js/app/viewmodels/terminal.js b/src/octoprint/static/js/app/viewmodels/terminal.js
index 4c61d32e..cee5f4f8 100644
--- a/src/octoprint/static/js/app/viewmodels/terminal.js
+++ b/src/octoprint/static/js/app/viewmodels/terminal.js
@@ -305,6 +305,10 @@ $(function() {
             }
         };
 
+        self.copyAll = function() {
+            copyToClipboard(self.plainLogLines().join("\n"));
+        };
+
         // command matching regex
         // (Example output for inputs G0, G1, G28.1, M117 test)
         // - 1: code including optional subcode. Example: G0, G1, G28.1, M117
diff --git a/src/octoprint/templates/tabs/terminal.jinja2 b/src/octoprint/templates/tabs/terminal.jinja2
index ea351e5c..df576260 100644
--- a/src/octoprint/templates/tabs/terminal.jinja2
+++ b/src/octoprint/templates/tabs/terminal.jinja2
@@ -7,9 +7,17 @@
             <a class="btn add-on" id="terminal-send" data-bind="click: sendCommand, enable: isOperational() && loginState.isUser()">{{ _('Send') }}</a>
         </div>
     </div>
-    <small class="pull-left" data-bind="visible: fancyFunctionality()"><button class="btn btn-mini" data-bind="click: toggleAutoscroll, css: {active: autoscrollEnabled}">{{ _('Autoscroll') }}</button> <span data-bind="text: lineCount, visible: enableFancyFunctionality"></span></small>
-    <small class="pull-right" data-bind="visible: fancyFunctionality()"><a href="#" data-bind="click: scrollToEnd">{{ _("Scroll to end") }}</a>&nbsp;|&nbsp;<a href="#" data-bind="click: selectAll">{{ _("Select all") }}</a></small>
-    <small class="pull-left text-warning" data-bind="visible: !fancyFunctionality()" style="display: none">{{ _('For performance reasons only a limited amount of terminal functionality is enabled right now.') }}</small>
+    <small class="pull-left" data-bind="visible: fancyFunctionality()">
+        <button class="btn btn-mini" data-bind="click: toggleAutoscroll, css: {active: autoscrollEnabled}">{{ _('Autoscroll') }}</button>
+        <span data-bind="text: lineCount, visible: enableFancyFunctionality"></span>
+        <span data-bind="visible: !autoscrollEnabled() && enableFancyFunctionality()" style="display: none">(<a href="javascript:void(0)" data-bind="click: scrollToEnd">{{ _("Scroll to end") }}</a>)</span>
+    </small>
+    <small class="pull-right">
+        <a href="javascript:void(0)" data-bind="click: copyAll"><i class="fa fa-copy" title="{{ _('Copy all') }}"></i> {{ _('Copy all') }}</a>
+    </small>
+    <small class="pull-left text-warning" data-bind="visible: !fancyFunctionality()" style="display: none">
+        {{ _('For performance reasons only a limited amount of terminal functionality is enabled right now.') }}
+    </small>
 </div>
 
 <div class="row-fluid">