Index: chrome/browser/resources/contextual_search/promo.css |
diff --git a/chrome/browser/resources/contextual_search/promo.css b/chrome/browser/resources/contextual_search/promo.css |
index 098ab121c54bdf1e284c50f4adad45d8f474e01f..63fe3392ee401b7ad2ca51b0702e639840192146 100644 |
--- a/chrome/browser/resources/contextual_search/promo.css |
+++ b/chrome/browser/resources/contextual_search/promo.css |
@@ -26,7 +26,6 @@ |
} |
body { |
- color: #414141; |
font-family: 'Roboto2', sans-serif; |
margin: 0; |
} |
@@ -39,7 +38,21 @@ a.colored-link { |
color: rgb(66, 133, 244); |
} |
+#container { |
+ /* NOTE(pedrosimonetti): There's an implicit extra top margin that is |
+ * rendered natively (currently using 24dp). So, the total padding will |
+ * be 38dp (24dp + 14dp). For more info, see SEARCH_BAR_HEIGHT_STATE_PROMO |
+ * in ContextualSearchPanelBase.java. |
+ * |
+ * We're also setting the side and bottom paddings to ensure to make sure |
+ * that when computing the height of the container all margins/paddings will |
+ * be considered. |
+ */ |
+ padding: 14px 16px 12px; |
+} |
+ |
#button-container { |
+ margin-top: 24px; |
text-align: end; |
width: 100%; |
} |
@@ -52,21 +65,24 @@ a.colored-link { |
} |
#description { |
+ color: #7E7E7E; |
font-size: 16px; |
- line-height: 1.4em; |
- margin: 24px 16px 4px 16px; |
+ line-height: 1.38em; |
+ margin: 12px 0 24px; |
} |
/* Some properties below can be overridden in landscape orientation. */ |
#heading { |
- font-size: 24px; |
- letter-spacing: 1px; |
- margin-top: 0; |
+ font-size: 23px; |
+ margin: 20px 0 12px; |
text-align: center; |
} |
-#header-image { |
- height: 100px; |
- margin: 0 auto 24px auto; |
+.header-image { |
+ background-image: url(header.svg); |
+ background-repeat: no-repeat; |
+ height: 98px; |
+ margin: 0 auto 38px auto; |
+ width: 156px; |
} |
.portrait { |
display: block; |
@@ -78,10 +94,27 @@ a.colored-link { |
/* Landscape */ |
@media screen and (orientation:landscape) { |
#heading { |
- padding-top: 16px; |
+ margin-top: 0; |
+ /* The heading text and description text should be aligned, therefore |
+ * the left margin here will be equal to the header image width (156px) |
+ * plus its right margin (24px). Therefore the total left should be |
+ * 156px + 24px = 180px. |
+ */ |
+ margin-left: 180px; |
+ padding-top: 8px; |
+ text-align: left; |
} |
- #header-image { |
- margin: 0 24px; |
+ .header-image { |
+ /* The header image is supposed to be vertically centered when the promo |
+ * is in landscape mode. For now, we're forcefully moving the image 4 |
+ * pixels up to make it centered. A better approach would be using CSS |
+ * flexbox to properly center it, but this will require changing the |
+ * markup and styling of the whole promo, and it could be tricky coming |
+ * up with a single markup that works in both portrait and lanscape modes. |
+ */ |
+ margin: 0 24px 0 0; |
+ position: relative; |
+ top: -4px; |
} |
.portrait { |
display: none; |
@@ -101,11 +134,12 @@ button { |
display: inline-block; |
font-family: 'Roboto2', sans-serif; |
font-size: 14px; |
- margin: 14px 0; |
+ margin: 6px 0; |
/* We use a slightly different top-bottom padding because Roboto has a |
- rendering bug which makes an extra padding to be rendered at the bottom of |
- text. */ |
- padding: 17px 14px 13px 14px; |
+ * rendering bug which makes an extra padding to be rendered at the bottom of |
+ * text. |
+ */ |
+ padding: 12px 16px 8px; |
white-space: nowrap; |
} |
@@ -117,7 +151,6 @@ button .caption { |
background: rgb(66, 133, 244); |
background-clip: padding-box; |
border-radius: 3px; |
- margin-right: 24px; |
} |
#optin-button .caption { |