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

Unified Diff: chrome/browser/resources/ntp4/apps_page.css

Issue 12985002: Add a new App launcher promo to the apps page / NTP. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: OWNERS review 2. Created 7 years, 9 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/ntp4/apps_page.css
diff --git a/chrome/browser/resources/ntp4/apps_page.css b/chrome/browser/resources/ntp4/apps_page.css
index dbea2ba6480dfe78b00f9bbc2ca5a6bc8579ea5e..4788259abcd60ff0de98c155d2dccb9c0f4a9128 100644
--- a/chrome/browser/resources/ntp4/apps_page.css
+++ b/chrome/browser/resources/ntp4/apps_page.css
@@ -99,3 +99,66 @@
.app .invisible {
visibility: hidden;
}
+
+.app-launcher-promo {
+ background-color: white;
+ border: 1px solid lightgray;
+ border-bottom-width: 3px;
+ border-radius: 2px;
+ border-top-width: 2px;
+ bottom: 90px;
+ font-family: Arial, Helvetica, sans-serif;
arv (Not doing code reviews) 2013/03/26 14:02:39 Remove font-family
MAD 2013/03/26 15:42:43 If I do, I get the wrong font. 'Segoe UI', Tahoma,
arv (Not doing code reviews) 2013/03/26 15:53:34 So, we don't want the promo to have the same font
MAD 2013/03/26 17:02:46 I was just following the spec, I'll mention it on
+ height: 120px;
+ left: 50%;
+ margin-left: -300px;
+ position: fixed;
arv (Not doing code reviews) 2013/03/26 14:02:39 I'm concerned about this one too. The notification
MAD 2013/03/26 15:42:43 Yes, the specs is to have this fixed above the foo
+ width: 600px;
+}
+
+.app-launcher-promo > .close-button {
+ position: absolute;
+ right: 10px;
+ top: 10px;
+ width: 14px;
+}
+
+.apps-promo-text {
+ color: #222;
+ font-family: inherit;
arv (Not doing code reviews) 2013/03/26 14:02:39 Is this needed?. inherit is the default. Maybe som
MAD 2013/03/26 15:42:43 Done.
+ font-size: 16px;
+ left: 30px;
+ line-height: 24px;
+ position: absolute;
+ top: 30px;
+}
+
+.apps-promo-learn-more {
+ background-color: rgb(77, 144, 254);
+ border: 1px solid rgb(47, 91, 183);
+ border-radius: 2px;
+ color: white;
+ cursor: default;
arv (Not doing code reviews) 2013/03/26 14:02:39 If it is a link, why don't we want it to show the
MAD 2013/03/26 15:42:43 Again, this is part of the internal spec I was fol
arv (Not doing code reviews) 2013/03/26 15:53:34 Done.
+ font-family: inherit;
+ font-size: 11px;
+ font-weight: bold;
+ height: 27px;
+ left: 30px;
+ line-height: 27px;
+ padding: 0 8px;
+ position: absolute;
+ text-align: center;
+ top: 70px;
+ width: 90px;
+}
+
+.apps-promo-learn-more:hover {
+ background-image: -webkit-linear-gradient(
+ top, rgb(77, 144, 254), rgb(53, 122, 232));
+ border: 1px solid rgb(47, 91, 183);
+}
+
+.app-launcher-promo > img {
+ bottom: 0;
+ position: absolute;
+ right: 30px;
+}

Powered by Google App Engine
This is Rietveld 408576698