Add the ability to filter by time to task list.

Add the ability to filter by time to task list.

Also optimize the task-list to look better at 1200px width.

BUG=684563
Review-Url: https://codereview.chromium.org/2696613007
Committed: https://github.com/luci/luci-py/commit/ff0ee7ec014926370e98e576f767f0381d18c2c1

[kjlubick, 2017-02-16 15:39:59]

kjlubick@google.com changed reviewers:
+ maruel@chromium.org, stephana@chromium.org

[kjlubick, 2017-02-16 15:40:00]

See it live at 
https://2637-d8a3c8d-tainted-kjlubick-dot-chromium-swarm-dev.appspot.com/task...

Screencast:
https://screencast.googleplex.com/cast/NTkxMTkzMjY1NzcyOTUzNnwzNTNlNzE4ZC1lYQ

Here's a screenshot at 1200px:
https://screenshot.googleplex.com/0BjUfOxjXcO

[stephana-chromium, 2017-02-16 18:32:03]

https://codereview.chromium.org/2696613007/diff/40001/appengine/swarming/ui/r...
File appengine/swarming/ui/res/imp/tasklist/task-filters.html (right):

https://codereview.chromium.org/2696613007/diff/40001/appengine/swarming/ui/r...
appengine/swarming/ui/res/imp/tasklist/task-filters.html:232: ts = ts || "1";
Nit: _parseTS would make it clearer that this is a private/internal function.

https://codereview.chromium.org/2696613007/diff/40001/appengine/swarming/ui/r...
appengine/swarming/ui/res/imp/tasklist/task-filters.html:437: setEndTS:
function(ts) {
If these are part of the API they should be documented.
If they are not they should have a "_" prefix. Same for functions below.

[kjlubick, 2017-02-17 14:13:33]

https://codereview.chromium.org/2696613007/diff/40001/appengine/swarming/ui/r...
File appengine/swarming/ui/res/imp/tasklist/task-filters.html (right):

https://codereview.chromium.org/2696613007/diff/40001/appengine/swarming/ui/r...
appengine/swarming/ui/res/imp/tasklist/task-filters.html:232: ts = ts || "1";
On 2017/02/16 at 18:32:03, stephana-chromium wrote:
> Nit: _parseTS would make it clearer that this is a private/internal function.

Done.

https://codereview.chromium.org/2696613007/diff/40001/appengine/swarming/ui/r...
appengine/swarming/ui/res/imp/tasklist/task-filters.html:437: setEndTS:
function(ts) {
On 2017/02/16 at 18:32:03, stephana-chromium wrote:
> If these are part of the API they should be documented.
> If they are not they should have a "_" prefix. Same for functions below.

Done.

[stephana, 2017-02-17 14:35:03]

stephana@google.com changed reviewers:
+ stephana@google.com

[stephana, 2017-02-17 14:35:04]

LGTM

[KevinL, 2017-02-17 14:38:16]

kjlubick@chromium.org changed reviewers:
+ kjlubick@chromium.org

[KevinL, 2017-02-17 14:38:16]

The CQ bit was checked by kjlubick@chromium.org

[KevinL, 2017-02-17 14:38:17]

lgtm

[commit-bot: I haz the power, 2017-02-17 14:38:24]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-17 14:42:02]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1487342296577230,
"parent_rev": "4ac1c3735889aa3e743cf2d9ad23f65998a87f73", "commit_rev":
"ff0ee7ec014926370e98e576f767f0381d18c2c1"}

[commit-bot: I haz the power, 2017-02-17 14:42:06]

Description was changed from

==========
Add the ability to filter by time to task list.

Also optimize the task-list to look better at 1200px width.

BUG=684563
==========

to

==========
Add the ability to filter by time to task list.

Also optimize the task-list to look better at 1200px width.

BUG=684563

Review-Url: https://codereview.chromium.org/2696613007
Committed:
https://github.com/luci/luci-py/commit/ff0ee7ec014926370e98e576f767f0381d18c2c1
==========

[commit-bot: I haz the power, 2017-02-17 14:42:07]

Committed patchset #4 (id:60001) as
https://github.com/luci/luci-py/commit/ff0ee7ec014926370e98e576f767f0381d18c2c1