Index: chrome/browser/resources/vr_shell/vr_shell_ui.css |
diff --git a/chrome/browser/resources/vr_shell/vr_shell_ui.css b/chrome/browser/resources/vr_shell/vr_shell_ui.css |
index 66774d091860eec3fe86b0e2e71ecba881c330a4..ac008e08e2473d3802fdffc4f0ffe90567e559e7 100644 |
--- a/chrome/browser/resources/vr_shell/vr_shell_ui.css |
+++ b/chrome/browser/resources/vr_shell/vr_shell_ui.css |
@@ -4,7 +4,7 @@ |
html { |
background-color: rgba(255, 255, 255, 0.2); |
- font-size: 20px; |
+ font-family: 'Roboto', sans-serif; |
Dan Beam
2016/10/03 18:12:48
can you use text_defaults.css or text_defaults_md.
klausw
2016/10/03 19:47:30
Done, and thank you. I got this working with a bit
|
} |
.ui-button { |
Dan Beam
2016/10/03 18:12:48
this is a reallly generic name, so it's hard to un
klausw
2016/10/03 19:47:30
This CSS is for the pre-existing preliminary navig
|
@@ -15,3 +15,71 @@ html { |
text-align: center; |
vertical-align: middle; |
} |
+ |
+.webvr-message-box { |
+ align-items: center; |
+ background: transparent; |
Dan Beam
2016/10/03 18:12:48
why do you need this background: rule?
klausw
2016/10/03 19:47:31
Removed.
|
+ display: flex; |
+ flex-direction: column; |
+ font-weight: 400; |
Dan Beam
2016/10/03 18:12:48
which font-weight are you overriding? note, 400 =
klausw
2016/10/03 19:47:31
Removed. This used to set font-weight: 300 in an e
|
+ justify-content: center; |
+ position: absolute; |
+} |
+ |
+#webvr-not-secure-permanent { |
+ height: 128px; |
Dan Beam
2016/10/03 18:12:48
what's the natural size of the svg?
klausw
2016/10/03 19:47:31
The SVG is 36px x 36px and is used at that size. T
|
+ left: 0; |
Dan Beam
2016/10/03 18:12:48
i noticed you mentioned RTL earlier: are you sure
klausw
2016/10/03 19:47:31
This is intentionally text direction independent.
|
+ top: 0; |
+ width: 512px; |
+} |
+ |
+#webvr-not-secure-permanent .webvr-not-secure-icon { |
+ display: inline-block; |
+ margin: 20px 0; |
+ vertical-align: middle; |
+} |
+ |
+#webvr-not-secure-permanent .webvr-string { |
+ display: inline-block; |
+ margin: 20px 10.5px; |
Dan Beam
2016/10/03 18:12:48
why .5px?
klausw
2016/10/03 19:47:30
I was trying to exactly replicate the UI review ap
|
+ vertical-align: middle; |
+} |
+ |
+#webvr-not-secure-permanent .webvr-box { |
+ background-color: white; |
+ border-radius: 6px; |
+ box-shadow: 0 0 20px rgba(0,0,0,0.5); |
Dan Beam
2016/10/03 18:12:48
spaces after commas (i.e. rgba(0, 0, 0, 0.5);)
klausw
2016/10/03 19:47:31
Done.
|
+ box-sizing: border-box; |
+ color: #444; |
+ font-size: 26px; |
+ height: 78px; |
+ max-width: 472px; |
+ min-width: 226px; |
+ overflow: hidden; |
+ padding: 0 20px; |
+ white-space: nowrap; |
+} |
+ |
+#webvr-not-secure-transient { |
+ height: 256px; |
+ left: 0; |
+ top: 128px; |
+ width: 512px; |
+} |
+ |
+#webvr-not-secure-transient > div { |
+ background-color: rgba(26, 26, 26, 0.8); |
Dan Beam
2016/10/03 18:19:19
also note that this UI looks very much like a mate
klausw
2016/10/03 19:47:31
I'll take this up with the UI designer, I'm reluct
|
+ border-radius: 6px; |
+ box-sizing: border-box; |
Dan Beam
2016/10/03 18:12:49
why do you need to change the box-sizing?
klausw
2016/10/03 19:47:31
AFAIK the goal here was to ensure that the rounded
|
+ color: white; |
+ display: flex; |
+ flex-direction: column; |
+ font-size: 26px; |
+ justify-content: center; |
+ line-height: 1.4; |
+ max-height: 210px; |
+ min-height: 160px; |
+ padding: 20px; |
+ text-align: center; |
+ width: 512px; |
Dan Beam
2016/10/03 18:12:48
i don't understand where any of these widths come
klausw
2016/10/03 19:47:31
The width: 512px matches the absolutely positioned
|
+} |