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

Side by Side 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 /* Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 /* Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 * Use of this source code is governed by a BSD-style license that can be 2 * Use of this source code is governed by a BSD-style license that can be
3 * found in the LICENSE file. */ 3 * found in the LICENSE file. */
4 4
5 .app { 5 .app {
6 outline: none; 6 outline: none;
7 position: absolute; 7 position: absolute;
8 text-align: center; 8 text-align: center;
9 } 9 }
10 10
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
92 cursor: pointer; 92 cursor: pointer;
93 } 93 }
94 94
95 .app-img-container > img:first-child { 95 .app-img-container > img:first-child {
96 display: block; 96 display: block;
97 } 97 }
98 98
99 .app .invisible { 99 .app .invisible {
100 visibility: hidden; 100 visibility: hidden;
101 } 101 }
102
103 .app-launcher-promo {
104 background-color: white;
105 border: 1px solid lightgray;
106 border-bottom-width: 3px;
107 border-radius: 2px;
108 border-top-width: 2px;
109 bottom: 90px;
110 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
111 height: 120px;
112 left: 50%;
113 margin-left: -300px;
114 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
115 width: 600px;
116 }
117
118 .app-launcher-promo > .close-button {
119 position: absolute;
120 right: 10px;
121 top: 10px;
122 width: 14px;
123 }
124
125 .apps-promo-text {
126 color: #222;
127 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.
128 font-size: 16px;
129 left: 30px;
130 line-height: 24px;
131 position: absolute;
132 top: 30px;
133 }
134
135 .apps-promo-learn-more {
136 background-color: rgb(77, 144, 254);
137 border: 1px solid rgb(47, 91, 183);
138 border-radius: 2px;
139 color: white;
140 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.
141 font-family: inherit;
142 font-size: 11px;
143 font-weight: bold;
144 height: 27px;
145 left: 30px;
146 line-height: 27px;
147 padding: 0 8px;
148 position: absolute;
149 text-align: center;
150 top: 70px;
151 width: 90px;
152 }
153
154 .apps-promo-learn-more:hover {
155 background-image: -webkit-linear-gradient(
156 top, rgb(77, 144, 254), rgb(53, 122, 232));
157 border: 1px solid rgb(47, 91, 183);
158 }
159
160 .app-launcher-promo > img {
161 bottom: 0;
162 position: absolute;
163 right: 30px;
164 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698