improve LDtrait bp window validation behavior

CBIIT · Apr 24, 2020 · b511206 · b511206
1 parent 920bcf6
commit b511206
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/LDlink/LDlink-4.1.0.js b/LDlink/LDlink-4.1.0.js
@@ -4235,14 +4235,12 @@ function validateLDtraitBasePairWindow() {
     var currentValue = $(textarea).val();
     var currentValueNoCommas = currentValue.replace(/\,/g, '');
     var hasError = !currentValue.match(pattern) || (currentValueNoCommas < 0 || currentValueNoCommas > 1000000);
-    console.log('hasError', hasError);
     $(textarea).toggleClass('error', hasError);
     $(textarea).toggleClass('ok', !hasError);
     if (hasError) {
         $(textarea).attr('title', errorMsg);
         return false;
     } else {
-        // $(textarea).removeAttr('title');
         return true;
     }
 }

diff --git a/LDlink/index.html b/LDlink/index.html
@@ -1690,9 +1690,9 @@ <h2 class="text-center">LDtrait Tool<sup><a class="help-anchor-link glyphicon gl
 									</div>
 									<div style="clear:both;"></div>
 									<div class="input-group row" style="min-width: 200px;">
-										<label id="ldtrait-bp-window-label" for="ldtrait-bp-window" style="width: 10px">&plusmn;</label>
-										<input id="ldtrait-bp-window" value="500000" type="text" pattern="^(\d+|\d{1,3}(,\d{3})*)(\.\d+)?$"
-											style="width: 70px;" title="Value must be a number between 0 and 1,000,000" required/>
+										<label id="ldtrait-bp-window-label" for="ldtrait-bp-window" style="width: 5px">&plusmn;</label>
+										<input id="ldtrait-bp-window" value="500000" type="number" pattern="^(\d+|\d{1,3}(,\d{3})*)(\.\d+)?$"
+											min="0" max="1000000" style="width: 75px;" title="Value must be a number between 0 and 1,000,000" required/>
 										<span>base pair window</span>
 									</div>
 									<div style="clear:both;"></div>