diff options
author | kelvinp@chromium.org <kelvinp@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-03-29 17:48:04 +0000 |
---|---|---|
committer | kelvinp@chromium.org <kelvinp@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-03-29 17:48:04 +0000 |
commit | dc87ae4463aa9cf5a167f0f999d04981f1e041b9 (patch) | |
tree | 3b835ab8ee6a991edf2e270b570cb8a5e661478a /remoting/webapp | |
parent | 93d798a4360e761db15a12db5c856b8bce6d8094 (diff) | |
download | chromium_src-dc87ae4463aa9cf5a167f0f999d04981f1e041b9.zip chromium_src-dc87ae4463aa9cf5a167f0f999d04981f1e041b9.tar.gz chromium_src-dc87ae4463aa9cf5a167f0f999d04981f1e041b9.tar.bz2 |
Fix Host stuck at the right side
Cause:
This bug only repro on chrome v33 and v34
<div class="vertically-centered">
<div class="horizontally-centered">
<div id="video-container">
<video id="mediasource-video-output"></video>
<div id="client-plugin-container"></div>
</div>
</div>
</div>
Due to a chrome layout bug, the width of video-container collapses to 0 when the zoom level is greater than 100%. As a result, the flexbox defined in horizonally-centered fails to center the element.
Fix:
Set width=100% on video-container so that its width remains valid. Since video-container now sizes to the parent, we need to move the horizontal-centered flexbox to the children of video-container.
BUG=357114
Review URL: https://codereview.chromium.org/217363002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@260386 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'remoting/webapp')
-rw-r--r-- | remoting/webapp/html/client_plugin.html | 4 | ||||
-rw-r--r-- | remoting/webapp/main.css | 1 |
2 files changed, 2 insertions, 3 deletions
diff --git a/remoting/webapp/html/client_plugin.html b/remoting/webapp/html/client_plugin.html index 628c6fa..af375ae 100644 --- a/remoting/webapp/html/client_plugin.html +++ b/remoting/webapp/html/client_plugin.html @@ -4,10 +4,8 @@ Use of this source code is governed by a BSD-style license that can be found in the LICENSE file. --> <div class="vertically-centered"> - <div class="horizontally-centered"> <div id="video-container"> <video id="mediasource-video-output"></video> - <div id="client-plugin-container"></div> + <div id="client-plugin-container" class="horizontally-centered"></div> </div> - </div> </div> diff --git a/remoting/webapp/main.css b/remoting/webapp/main.css index 7b50dbb..2eb0948 100644 --- a/remoting/webapp/main.css +++ b/remoting/webapp/main.css @@ -743,6 +743,7 @@ html.apps-v2.scrollable { /* video-container needs relative position so that mediasource-video-output can * be positioned relative to the parent with position:absolute. */ #video-container { + width: 100%; position: relative; } |