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

Unified Diff: chrome/browser/resources/set_as_default_browser.css

Issue 10837084: The updated (simplified) look of set-as-default dialog on Windows8. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed up css/html to meet the new design. Created 8 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: chrome/browser/resources/set_as_default_browser.css
diff --git a/chrome/browser/resources/set_as_default_browser.css b/chrome/browser/resources/set_as_default_browser.css
index b947c4db16a24b25cb5971001c2b13735f6706a6..c86796fe927230eca70e7ca33ce452ad01bd2ef1 100644
--- a/chrome/browser/resources/set_as_default_browser.css
+++ b/chrome/browser/resources/set_as_default_browser.css
@@ -6,14 +6,13 @@
#metro-setup-outer-container {
-webkit-box-align: center;
-webkit-box-orient: vertical;
- -webkit-box-pack: center;
+ -webkit-box-pack: start;
-webkit-transition: 200ms opacity;
background-color: rgba(255, 255, 255, 0.75);
bottom: 0;
display: -webkit-box;
left: 0;
overflow: auto;
- padding: 20px;
position: fixed;
right: 0;
top: 0;
@@ -28,18 +27,17 @@
background-color: white;
color: #333;
display: -webkit-box;
- margin-bottom: 6px;
- margin-top: 6px;
min-width: 400px;
- padding: 0;
+ padding-top: 70pt;
James Hawkins 2012/08/03 22:51:44 pt? We don't usually use pt, so can you use px in
motek. 2012/08/06 18:38:30 Done.
position: relative;
- width: 400px;
+ width: 500px;
}
#metro-setup-outer-container .page .content-area {
-webkit-box-flex: 1;
+ margin: 0;
overflow: auto;
- padding: 6px 17px 26px;
+ padding: 0;
}
#metro-setup-overlay {
@@ -57,20 +55,22 @@
#metro-setup-outer-container .page h1 {
border-bottom: 0;
- color: rgb(34, 34, 34);
- font-size: 340%;
+ color: #323232;
+ font-size: 28pt;
James Hawkins 2012/08/03 22:51:44 font-size needs to be relative, not absolute.
motek. 2012/08/06 18:38:30 Done.
font-weight: normal;
margin: 0;
padding-bottom: 0;
- padding-top: 13px;
+ padding-top: 0;
text-align: center;
}
#metro-setup-outer-container .page h2 {
border-bottom: 0;
- color: rgb(102, 102, 102);
- font-size: 160%;
+ color: #787878;
+ font-size: 14pt;
James Hawkins 2012/08/03 22:51:44 Relative, not absolute.
motek. 2012/08/06 18:38:30 Done.
font-weight: normal;
+ margin-bottom: 4.9em;
James Hawkins 2012/08/03 22:51:44 Any reason we're using em here and px elsewhere?
motek. 2012/08/06 18:38:30 Not really. I mean: there was a reason. I got a mo
+ margin-top: 0.75em;
text-align: center;
}
@@ -146,11 +146,14 @@
z-index: 4 !important;
}
-#metro-action-box #return-button {
- -webkit-margin-end: auto;
- -webkit-margin-start: 30px;
+.button-strip {
James Hawkins 2012/08/03 22:51:44 Is this the same button-strip class used in the op
motek. 2012/08/06 18:38:30 No, it is a coincidence. I needed a container to h
+ margin: 1px;
+ text-align: center;
}
-.button-strip {
- padding: 7px;
+#chrome-logo-box {
+ bottom: 23pt;
James Hawkins 2012/08/03 22:51:44 px
motek. 2012/08/06 18:38:30 Done.
+ position: absolute;
+ text-align: center;
+ width: 100%;
}

Powered by Google App Engine
This is Rietveld 408576698