Changed the highlight color to be a less abrasive yellow as per

Changed the highlight color to be a less abrasive yellow as per
M-A Ruel's suggestion on cl 2921983002

BUG=727947
Review-Url: https://codereview.chromium.org/2930813003
Committed: https://github.com/luci/luci-py/commit/d0e96b62aa25eae0a3660f0390ed6baef8abffda

[cwpayton, 2017-06-08 17:03:19]

cwpayton@google.com changed reviewers:
+ hinoka@google.com, sergeyberezin@google.com

[cwpayton, 2017-06-08 17:03:19]

Marc-Antoine noted that the highlight color was too loud for the viewer, and
suggested a lighter color that was more subtle and easier to look at, so I
changed the shade of the highlight.

[Ryan Tseng, 2017-06-08 17:47:15]

Can you attach a screenshot (for this, and all other UI related CLs),
re-vulcanize the files, and reupload?

[cwpayton, 2017-06-08 17:49:23]

Description was changed from

==========
Changed the highlight color to be a less abrasive yellow as per
M-A Ruel's suggestion on cl 2921983002

BUG=727947
==========

to

==========
Changed the highlight color to be a less abrasive yellow as per
M-A Ruel's suggestion on cl 2921983002

Screenshot: https://screenshot.googleplex.com/WmHWFwjmnZD

BUG=727947
==========

[cwpayton, 2017-06-08 18:10:17]

PTAL

[Ryan Tseng, 2017-06-08 18:14:09]

Put the screenshot as a message reply, not in the description.  Because this is
going to end up on github.com and screenshots are mostly for reviewers.  (Also
it's a googleplex.com domain)

[cwpayton, 2017-06-08 18:17:54]

Description was changed from

==========
Changed the highlight color to be a less abrasive yellow as per
M-A Ruel's suggestion on cl 2921983002

Screenshot: https://screenshot.googleplex.com/WmHWFwjmnZD

BUG=727947
==========

to

==========
Changed the highlight color to be a less abrasive yellow as per
M-A Ruel's suggestion on cl 2921983002

BUG=727947
==========

[cwpayton, 2017-06-08 18:18:07]

On 2017/06/08 18:14:09, Ryan Tseng wrote:
> Put the screenshot as a message reply, not in the description.  Because this
is
> going to end up on http://github.com and screenshots are mostly for reviewers.
 (Also
> it's a http://googleplex.com domain)

Screenshot: https://screenshot.googleplex.com/WmHWFwjmnZD

[Sergey Berezin, 2017-06-08 18:37:44]

sergeyberezin@chromium.org changed reviewers:
+ maruel@chromium.org, sergeyberezin@chromium.org

[Sergey Berezin, 2017-06-08 18:37:45]

+maruel for the final verdict (since he asked for it).

LGTM on my side - thanks!

[M-A Ruel, 2017-06-08 18:43:03]

On 2017/06/08 18:37:45, Sergey Berezin wrote:
> +maruel for the final verdict (since he asked for it).
> 
> LGTM on my side - thanks!

I'm still not sure it's worth having the colored non-white background for these
but this is better, so lgtm.

[cwpayton, 2017-06-08 18:43:33]

The CQ bit was checked by cwpayton@google.com

[commit-bot: I haz the power, 2017-06-08 18:43:44]

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-06-08 18:46:56]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1496947413220400,
"parent_rev": "d3a810515f73c543eab2c1eb9db38a65f9d712e1", "commit_rev":
"d0e96b62aa25eae0a3660f0390ed6baef8abffda"}

[commit-bot: I haz the power, 2017-06-08 18:46:59]

Description was changed from

==========
Changed the highlight color to be a less abrasive yellow as per
M-A Ruel's suggestion on cl 2921983002

BUG=727947
==========

to

==========
Changed the highlight color to be a less abrasive yellow as per
M-A Ruel's suggestion on cl 2921983002

BUG=727947

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

[commit-bot: I haz the power, 2017-06-08 18:47:01]

Committed patchset #2 (id:20001) as
https://github.com/luci/luci-py/commit/d0e96b62aa25eae0a3660f0390ed6baef8abffda