Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(286)

Unified Diff: appengine/swarming/elements/res/imp/botlist/bot-filters.html

Issue 2338383002: Refactor prior to adding task-page (Closed) Base URL: git@github.com:luci/luci-py@master
Patch Set: Address nits Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: appengine/swarming/elements/res/imp/botlist/bot-filters.html
diff --git a/appengine/swarming/elements/res/imp/botlist/bot-filters.html b/appengine/swarming/elements/res/imp/botlist/bot-filters.html
index 810cff7b8882f2a216e2ae7a5f3912fda7198180..0c0b67b7ba6c329ba47ff277c225c734db5b002d 100644
--- a/appengine/swarming/elements/res/imp/botlist/bot-filters.html
+++ b/appengine/swarming/elements/res/imp/botlist/bot-filters.html
@@ -85,9 +85,9 @@
<url-param name="verbose"
value="{{verbose}}">
</url-param>
- <url-param name="limit"
+ <url-param name="_limit"
default_value="200"
- value="{{limit}}">
+ value="{{_limit}}">
</url-param>
<div class="container horizontal layout">
@@ -164,13 +164,13 @@ the fleet.">
<div class="side-by-side">
<paper-checkbox checked="{{verbose}}">Verbose Entries</paper-checkbox>
- <paper-input id="limit"
+ <paper-input id="_limit"
label="Limit Results"
auto-validate
min="0"
max="1000"
pattern="[0-9]+"
- value="{{limit}}">
+ value="{{_limit}}">
</paper-input>
</div>
</div>
@@ -224,7 +224,7 @@ the fleet.">
},
query_params: {
type: Object,
- computed: "_extractQueryParams(dimensions.*,_filters.*, limit)",
+ computed: "_extractQueryParams(dimensions.*,_filters.*, _limit)",
notify: true,
},
verbose: {
@@ -273,15 +273,15 @@ the fleet.">
}
}.bind(this));
params["dimensions"] = dims;
- var lim = Math.floor(this.limit)
+ var lim = parseInt(this._limit);
if (Number.isInteger(lim)) {
// Clamp the limit
lim = Math.max(lim, 1);
lim = Math.min(1000, lim);
- params["limit"] = [lim];
+ params["_limit"] = [lim];
// not !== because limit could be a string, e.g. "900"
- if (this.limit != lim) {
- this.set("limit", lim);
+ if (this._limit != lim) {
+ this.set("_limit", lim);
}
}
return params;
« no previous file with comments | « appengine/swarming/elements/build/js/js.js ('k') | appengine/swarming/elements/res/imp/botlist/bot-list.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698