[logdog-view] Update UX, fix bugs.

[logdog-view] Update UX, fix bugs.

Update the LogDog Web Viewer user experience and fix a bug where the
stream finishing wasn't always acknowledged by the UI.

This makes button visibility modal. While the stream is automatically
loading, control buttons will be hidden. When it is paused, they will be
shown. This will reduce the number of active buttons that the user has
to see during log interaction.

Following is no longer optional. It is enabled by default, and disabled
permanently for the duration of the stream on scroll or pause. This
removes the need for a user to manage this option.

BUG=None
TEST=local

Review-Url: https://codereview.chromium.org/2988993003
Committed: https://github.com/luci/luci-go/commit/810e5dc1dfbe94f009ae3cd5e0d3c92d1a375d88

[dnj, 2017-07-28 13:49:24]

formatting

[dnj, 2017-08-01 18:18:53]

dnj@chromium.org changed reviewers:
+ hinoka@chromium.org

[dnj, 2017-08-01 18:18:54]

PTAL

[dnj, 2017-08-01 18:26:28]

Example:
https://dev-dot-luci-logdog.appspot.com/v/?h=luci-logdog.appspot.com&s=chromi...

[Ryan Tseng, 2017-08-01 18:39:07]

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

[Ryan Tseng, 2017-08-01 18:39:07]

lgtm

https://codereview.chromium.org/2988993003/diff/100001/web/inc/logdog-stream-...
File web/inc/logdog-stream-view/view.ts (right):

https://codereview.chromium.org/2988993003/diff/100001/web/inc/logdog-stream-...
web/inc/logdog-stream-view/view.ts:183: handleMouseWheel() {
I'd probably only handle this if it is a scroll up event, and ignore scroll down
events.

[dnj, 2017-08-01 20:16:15]

https://codereview.chromium.org/2988993003/diff/100001/web/inc/logdog-stream-...
File web/inc/logdog-stream-view/view.ts (right):

https://codereview.chromium.org/2988993003/diff/100001/web/inc/logdog-stream-...
web/inc/logdog-stream-view/view.ts:183: handleMouseWheel() {
On 2017/08/01 18:39:07, Ryan Tseng wrote:
> I'd probably only handle this if it is a scroll up event, and ignore scroll
down
> events.

Done.

[dnj, 2017-08-01 20:16:18]

The CQ bit was checked by dnj@chromium.org

[dnj, 2017-08-01 20:16:19]

The patchset sent to the CQ was uploaded after l-g-t-m from hinoka@google.com
Link to the patchset: https://codereview.chromium.org/2988993003/#ps120001
(title: "unfollow only on up scroll")

[commit-bot: I haz the power, 2017-08-01 20:16: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-08-01 20:20:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-08-01 20:20:39]

Try jobs failed on following builders:
  Luci-go Win Tester on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/37b842e65f0df310)

[dnj, 2017-08-01 22:24:15]

The CQ bit was checked by dnj@chromium.org

[commit-bot: I haz the power, 2017-08-01 22:24:23]

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-08-01 22:31:18]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1501626255499300,
"parent_rev": "db21483023eebb29ad6bcf00585a6b854609c657", "commit_rev":
"810e5dc1dfbe94f009ae3cd5e0d3c92d1a375d88"}

[commit-bot: I haz the power, 2017-08-01 22:31:26]

Description was changed from

==========
[logdog-view] Update UX, fix bugs.

Update the LogDog Web Viewer user experience and fix a bug where the
stream finishing wasn't always acknowledged by the UI.

This makes button visibility modal. While the stream is automatically
loading, control buttons will be hidden. When it is paused, they will be
shown. This will reduce the number of active buttons that the user has
to see during log interaction.

Following is no longer optional. It is enabled by default, and disabled
permanently for the duration of the stream on scroll or pause. This
removes the need for a user to manage this option.

BUG=None
TEST=local
==========

to

==========
[logdog-view] Update UX, fix bugs.

Update the LogDog Web Viewer user experience and fix a bug where the
stream finishing wasn't always acknowledged by the UI.

This makes button visibility modal. While the stream is automatically
loading, control buttons will be hidden. When it is paused, they will be
shown. This will reduce the number of active buttons that the user has
to see during log interaction.

Following is no longer optional. It is enabled by default, and disabled
permanently for the duration of the stream on scroll or pause. This
removes the need for a user to manage this option.

BUG=None
TEST=local

Review-Url: https://codereview.chromium.org/2988993003
Committed:
https://github.com/luci/luci-go/commit/810e5dc1dfbe94f009ae3cd5e0d3c92d1a375d88
==========

[commit-bot: I haz the power, 2017-08-01 22:31:27]

Committed patchset #7 (id:120001) as
https://github.com/luci/luci-go/commit/810e5dc1dfbe94f009ae3cd5e0d3c92d1a375d88