Make isolate server look better on bigger screens

Make isolate server look better on bigger screens

Using container-fluid for the outside tells bootstrap we want to use as
much width as the screen has (minus some padding).

Additionally, we adjust the height of the iframe after loading to make
it expand to its full height, making things look nicer.

BUG=703581
BUG=698740
Review-Url: https://codereview.chromium.org/2759393004
Committed: https://github.com/luci/luci-py/commit/9e955a3cbcc9e019494ce7950497109bc31690b3

[kjlubick, 2017-03-21 17:47:59]

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

[kjlubick, 2017-03-21 17:48:00]

Before (on a 2560x1440 screen)
home page: https://screenshot.googleplex.com/MQXLdQTQQy6
text isolate: https://screenshot.googleplex.com/YUa91ANjm57
large isolated image: https://screenshot.googleplex.com/LyZJOnZnK4A

Before (on a 1280x720 screen)
home page: https://screenshot.googleplex.com/BL0R9aj2FoP
text isolate: https://screenshot.googleplex.com/BL0R9aj2FoP
large isolated image: https://screenshot.googleplex.com/mjWPAXu6WiV

Before Summary: It looks okay on medium-sized screens, but not so good on huge
screens.


After (on a 2560x1440 screen)
home page: https://screenshot.googleplex.com/nMZiypXDPwy
text isolate: https://screenshot.googleplex.com/hTNfCKwBu9a
large isolated image: https://screenshot.googleplex.com/7kA4UUOfmX3

After (on a 1280x720 screen)
home page: https://screenshot.googleplex.com/pMqBk6JgwoG
text isolate: https://screenshot.googleplex.com/R4kqV03gM5y
large isolated image: https://screenshot.googleplex.com/u9ysGCF2ZEf

After summary: It takes advantage of large screens.  The height correction on
the iframe also helps on small screens.


See it live at
https://2701-07473b8-tainted-kjlubick-dot-isolateserver-dev.appspot.com/brows...

[M-A Ruel, 2017-03-22 01:25:01]

partially rs lgtm

[kjlubick, 2017-03-22 11:56:09]

The CQ bit was checked by kjlubick@google.com

[commit-bot: I haz the power, 2017-03-22 11:56:13]

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-03-22 11:58:29]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490183769556570, "parent_rev":
"9a3698537d93b2fc03f1f1ab7797307d7728ab12", "commit_rev":
"9e955a3cbcc9e019494ce7950497109bc31690b3"}

[commit-bot: I haz the power, 2017-03-22 11:58:32]

Description was changed from

==========
Make isolate server look better on bigger screens

Using container-fluid for the outside tells bootstrap we want to use as
much width as the screen has (minus some padding).

Additionally, we adjust the height of the iframe after loading to make
it expand to its full height, making things look nicer.

BUG=703581
BUG=698740
==========

to

==========
Make isolate server look better on bigger screens

Using container-fluid for the outside tells bootstrap we want to use as
much width as the screen has (minus some padding).

Additionally, we adjust the height of the iframe after loading to make
it expand to its full height, making things look nicer.

BUG=703581
BUG=698740

Review-Url: https://codereview.chromium.org/2759393004
Committed:
https://github.com/luci/luci-py/commit/9e955a3cbcc9e019494ce7950497109bc31690b3
==========

[commit-bot: I haz the power, 2017-03-22 11:58:33]

Committed patchset #1 (id:1) as
https://github.com/luci/luci-py/commit/9e955a3cbcc9e019494ce7950497109bc31690b3