From da42713a191e2be8282c86172670d1c7a1943972 Mon Sep 17 00:00:00 2001
From: Oliver Hamlet <oliver.hamlet@gmail.com>
Date: Fri, 25 Nov 2016 21:10:08 +0000
Subject: [PATCH] Improve validation of editor's priority fields

prevent-invalid-input only applies to pattern matching, so ignore it.
Instead, auto-validate and provide an error message, and widen the
inputs so the message can be displayed fully.
---
 src/gui/html/elements/loot-plugin-editor.html | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/gui/html/elements/loot-plugin-editor.html b/src/gui/html/elements/loot-plugin-editor.html
index c8b4f1e9..37ecf8a5 100644
--- a/src/gui/html/elements/loot-plugin-editor.html
+++ b/src/gui/html/elements/loot-plugin-editor.html
@@ -83,6 +83,10 @@ loot-editor-close
       iron-pages > *:not(.iron-selected) {
         display: none;
       }
+      #priorityValue,
+      #globalPriorityValue {
+        width: 250px;
+      }
 
       /* Misc Styling. */
       paper-toolbar.medium-tall {
@@ -139,11 +143,11 @@ loot-editor-close
         </div>
         <div>
           <div>Priority</div>
-          <paper-input id="priorityValue" type="number" max="127" min="-127" step="1" prevent-invalid-input no-label-float></paper-input>
+          <paper-input id="priorityValue" error-message="Must be an integer between +/- 127 inclusive" type="number" max="127" min="-127" step="1" auto-validate no-label-float></paper-input>
         </div>
         <div>
           <div>Global Priority</div>
-          <paper-input id="globalPriorityValue" type="number" max="127" min="-127" step="1" prevent-invalid-input no-label-float></paper-input>
+          <paper-input id="globalPriorityValue" error-message="Must be an integer between +/- 127 inclusive" type="number" max="127" min="-127" step="1" auto-validate no-label-float></paper-input>
         </div>
       </div>
       <div id="after">