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

Unified Diff: chrome/browser/resources/ssl/interstitial_v2.css

Issue 319303003: Polish the new SSL interstitial (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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
Index: chrome/browser/resources/ssl/interstitial_v2.css
diff --git a/chrome/browser/resources/ssl/interstitial_v2.css b/chrome/browser/resources/ssl/interstitial_v2.css
index 2ea38bb5f3a4ff37eb5450996f707e822c63f9ab..e1ec1fe7470a91be1fcccfaa0f031afe108f171b 100644
--- a/chrome/browser/resources/ssl/interstitial_v2.css
+++ b/chrome/browser/resources/ssl/interstitial_v2.css
@@ -8,8 +8,8 @@ a {
body {
background: rgb(247, 247, 247);
- color: rgb(43, 43, 43);
- line-height: 1.7em;
+ color: rgb(88, 88, 88);
+ font-size: 16px;
}
button {
@@ -20,19 +20,36 @@ button {
color: #fff;
cursor: pointer;
float: right;
+ font-size: .875em;
+ height: 36px;
margin: -6px 0 0;
padding: 8px 24px;
}
+button:hover {
+ box-shadow: 0 1px 3px rgba(0, 0, 0, .50);
+}
+
+button:active {
+ background: rgb(50, 102, 213);
+}
+
#details {
- margin: 40px 0 150px 0;
+ color: rgb(105, 105, 105);
Dan Beam 2014/06/07 02:04:42 nit: use hex for rr == gg == bb
felt 2014/06/07 02:56:12 switched several to hex
+ margin: 45px 0 150px 0;
+}
+
+#details-button {
+ color: rgb(105, 105, 105);
+ font-size: .875em;
}
h1 {
- -webkit-margin-after: 15px;
+ -webkit-margin-after: 16px;
+ color: rgb(43, 43, 43);
font-size: 1.6em;
font-weight: normal;
- line-height: 1.2em;
+ line-height: 1.75em;
}
.hidden {
@@ -45,20 +62,22 @@ h1 {
url('images/2x/brokenssl_red.png') 2x);
background-repeat: no-repeat;
background-size: 100%;
- height: 69px;
- margin: 0 0 40px;
- width: 56px;
+ height: 72px;
+ margin: 0 0 45px;
+ width: 72px;
}
.interstitial-wrapper {
box-sizing: border-box;
+ font-size: 1em;
+ line-height: 1.6em;
margin: 10% auto 0;
- max-width: 552px;
+ max-width: 600px;
width: 100%;
}
.nav-wrapper {
- margin-top: 60px;
+ margin-top: 84px;
}
.nav-wrapper::after {
@@ -68,8 +87,8 @@ h1 {
width: 100%;
}
-p {
- margin-bottom: 22px;
+p.add-top-margin {
Dan Beam 2014/06/07 02:04:42 #details p:not(:first-of-type) { margin-top: 39p
felt 2014/06/07 02:56:12 Done.
+ margin-top: 39px;
}
@media (max-width: 700px) {

Powered by Google App Engine
This is Rietveld 408576698