Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(324)

Unified Diff: remoting/webapp/main.css

Issue 478033004: Fix a couple of scroll-related issues: (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: remoting/webapp/main.css
diff --git a/remoting/webapp/main.css b/remoting/webapp/main.css
index 822d6e2f349f2dadba0b374f31f02af8fe701dda..d9327d440092bd5a9158d5ce7a02992f3a4b698a 100644
--- a/remoting/webapp/main.css
+++ b/remoting/webapp/main.css
@@ -619,16 +619,18 @@ button {
top: 200px;
left: 0;
width: 100%;
+ z-index: 2;
}
.dialog-screen {
position: absolute;
- top: 0;
- left: 0;
- width: 100%;
- height: 100%;
+ top: 1px;
+ left: 1px;
+ width: calc(100% - 2px);
+ height: calc(100% - 2px);
Jamie 2014/08/23 00:12:00 Allow 2px for the apps v2 window border (which is
garykac1 2014/08/23 00:21:40 Doesn't this affect the v1 app border as well?
Jamie 2014/08/23 00:32:57 For the v1 app, there is no HTML window border, so
background-color: #fff;
opacity: 0.75;
+ z-index: 1;
}
/* TODO(jamiewalch): crbug.com/252796: Remove these once crbug.com/240772
@@ -747,6 +749,12 @@ html.apps-v2.scrollable {
width: 100%;
}
+/* Override full-height class until connected, otherwise it takes up 100% of
+ * the window height in addition to the home-screen UI. */
+body:not(.connected) #session-mode {
+ height: auto;
+}
+
/* video-container needs relative position so that mediasource-video-output can
* be positioned relative to the parent with position:absolute. */
#video-container {

Powered by Google App Engine
This is Rietveld 408576698