From 8ad165c62591483363712e6dd5218c43d4ea0f67 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Gina=20H=C3=A4u=C3=9Fge?= <gina@octoprint.org>
Date: Mon, 3 Jul 2017 16:08:41 +0200
Subject: [PATCH] Fix blur of temp input focus on submit

Also separated "clear offset" button in offset editor from input
field, looks better this way.
---
 src/octoprint/static/js/app/viewmodels/temperature.js | 3 ++-
 src/octoprint/templates/dialogs/temperature.jinja2    | 2 +-
 src/octoprint/templates/tabs/temperature.jinja2       | 4 ++--
 3 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/octoprint/static/js/app/viewmodels/temperature.js b/src/octoprint/static/js/app/viewmodels/temperature.js
index 828e1da8..6b7ac5d6 100644
--- a/src/octoprint/static/js/app/viewmodels/temperature.js
+++ b/src/octoprint/static/js/app/viewmodels/temperature.js
@@ -476,8 +476,9 @@ $(function() {
             }
         };
 
-        self.setTarget = function(item) {
+        self.setTarget = function(item, form) {
             var value = item.newTarget();
+            $(form).find("input").blur();
             if (value === undefined || (typeof(value) == "string" && value.trim() == "")) return OctoPrintClient.createRejectedDeferred();
             return self.setTargetToValue(item, value);
         };
diff --git a/src/octoprint/templates/dialogs/temperature.jinja2 b/src/octoprint/templates/dialogs/temperature.jinja2
index a1d765b8..78911009 100644
--- a/src/octoprint/templates/dialogs/temperature.jinja2
+++ b/src/octoprint/templates/dialogs/temperature.jinja2
@@ -20,8 +20,8 @@
                         <input type="number" min="-50" max="50" class="input-mini input-nospin" style="width: 30px" data-bind="value: changingOffset.newOffset, event: { focus: function(d, e) {$root.handleFocus(e, 'target', changingOffset.item);}, keyup: function(d, e) {$root.handleEnter(e, 'offset', changingOffset.item);} }">
                         <span class="add-on">&deg;C</span>
                         <button type="button" class="btn btn-input-inc" data-bind="click: $root.incrementChangeOffset" title="{{ _('+1') }}"><i class="fa fa-plus"></i></button>
-                        <button type="button" class="btn" data-bind="click: $root.deleteChangeOffset" title="{{ _('Delete Offset') }}"><i class="fa fa-trash"></i></button>
                     </div>
+                    <button type="button" class="btn" data-bind="click: $root.deleteChangeOffset" title="{{ _('Delete Offset') }}"><i class="fa fa-trash"></i></button>
                     <span class="help-block">{{ _('Hint: Hitting <kbd>Enter</kbd> in the input field will also submit the form') }}</span>
                 </div>
             </div>
diff --git a/src/octoprint/templates/tabs/temperature.jinja2 b/src/octoprint/templates/tabs/temperature.jinja2
index a64d9f88..bb8119d1 100644
--- a/src/octoprint/templates/tabs/temperature.jinja2
+++ b/src/octoprint/templates/tabs/temperature.jinja2
@@ -22,7 +22,7 @@
         <th class="temperature_tool" data-bind="text: name"></th>
         <td class="temperature_actual" data-bind="html: formatTemperature(actual())"></td>
         <td class="temperature_target">
-            <form class="form-inline" style="margin:0">
+            <form class="form-inline" style="margin:0" data-bind="submit: function(element) { $root.setTarget($data, element) }">
                 <div class="input-prepend input-append">
                     <button type="button" class="btn btn-input-dec" data-bind="click: $root.decrementTarget, enable: $root.isOperational() && $root.loginState.isUser()" title="{{ _('-1') }}"><i class="fa fa-minus"></i></button>
                     <input type="number" min="0" max="999" class="input-mini input-nospin" style="width: 30px" data-bind="attr: {placeholder: cleanTemperature(target())}, value: newTarget, valueUpdate: 'input', enable: $root.isOperational() && $root.loginState.isUser(), event: { focus: function(d, e) {$root.handleFocus(e, 'target', $data) } }">
@@ -30,7 +30,7 @@
                     <button type="button" class="btn btn-input-inc" data-bind="click: $root.incrementTarget, enable: $root.isOperational() && $root.loginState.isUser()" title="{{ _('+1') }}"><i class="fa fa-plus"></i></button>
                 </div>
                 <div class="btn-group">
-                    <button type="submit" data-bind="click: $root.setTarget, enable: $root.isOperational() && $root.loginState.isUser() && $data.newTargetValid()" class="btn btn-primary" title="{{ _('Set') }}"><i class="fa fa-check"></i></button>
+                    <button type="submit" data-bind="enable: $root.isOperational() && $root.loginState.isUser() && $data.newTargetValid()" class="btn btn-primary" title="{{ _('Set') }}"><i class="fa fa-check"></i></button>
                     <button type="button" class="btn btn-primary dropdown-toggle" data-toggle="dropdown" data-bind="enable: $root.isOperational() && $root.loginState.isUser()">
                         <span class="caret"></span>
                     </button>