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

Unified Diff: components/neterror/resources/neterror.css

Issue 2955153002: Add arcade mode to chrome://dino (Closed)
Patch Set: Fix incorrect scaling on wide screens and overflow Created 3 years, 6 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
« no previous file with comments | « no previous file | components/neterror/resources/offline.js » ('j') | components/neterror/resources/offline.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/neterror/resources/neterror.css
diff --git a/components/neterror/resources/neterror.css b/components/neterror/resources/neterror.css
index 4649cd82bbb7477aa6ca691e0e22171e2d131767..1e1286ad82ac4d4bffcd5b98d34b0c0f73c7d2ec 100644
--- a/components/neterror/resources/neterror.css
+++ b/components/neterror/resources/neterror.css
@@ -462,13 +462,6 @@ html[subframe] body {
}
}
-@media (min-width: 600px) and (max-width: 736px) and (orientation: landscape) {
mmenke 2017/07/05 18:51:28 Why are you removing this?
edwardjung 2017/07/06 19:33:19 Centre aligning the container makes more sense for
- .offline .interstitial-wrapper {
- margin-left: 0;
- margin-right: 0;
- }
-}
-
@media (min-width: 420px) and (max-width: 736px) and
(min-height: 240px) and (max-height: 420px) and
(orientation:landscape) {
@@ -514,3 +507,35 @@ html[subframe] body {
width: auto;
}
}
+
+.arcade-mode {
mmenke 2017/07/05 18:51:28 Is this needed? It seems like the next block appl
edwardjung 2017/07/06 19:33:19 Done.
+ overflow: hidden;
+}
+
+.arcade-mode,
+.arcade-mode .runner-container,
+.arcade-mode .runner-canvas {
+ max-width: 100%;
+ overflow: hidden;
+}
+
+.arcade-mode #buttons,
+.arcade-mode #main-content {
+ opacity: 0;
+ overflow: hidden;
mmenke 2017/07/05 18:51:29 I assume there's a reason we aren't just using dis
edwardjung 2017/07/06 19:33:19 With display none: you can't transition out the el
+}
+
+.arcade-mode .interstitial-wrapper {
+ height: 100vh;
mmenke 2017/07/05 18:51:29 vh? Is that a typo, or does it mean something?
edwardjung 2017/07/06 19:33:19 Not a typo, a new proportional unit referring to v
+ max-width: 100%;
+ overflow: hidden;
+}
+
+.arcade-mode .runner-container {
+ left: 0;
+ margin: auto;
+ right: 0;
+ transform-origin: top center;
+ transition: transform 250ms cubic-bezier(0.4, 0.0, 1, 1) .4s;
+ z-index: 2;
+}
« no previous file with comments | « no previous file | components/neterror/resources/offline.js » ('j') | components/neterror/resources/offline.js » ('J')

Powered by Google App Engine
This is Rietveld 408576698