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

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: FieldTrial CR Comments 1 + one more test fix. 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 height: 120px;
111 left: 50%;
112 margin-left: -300px;
113 position: fixed;
114 width: 600px;
115 }
116
117 .app-launcher-promo > .close-button {
118 position: absolute;
119 right: 10px;
120 top: 10px;
121 width: 14px;
122 }
123
124 .apps-promo-text {
125 color: #222;
126 font: 16px normal Arial, Helvetica, sans-serif;
arv (Not doing code reviews) 2013/03/25 15:04:38 Why does the font need to be reset? It is usually
MAD 2013/03/25 15:32:24 You mean set the font on the outer div and inherit
arv (Not doing code reviews) 2013/03/26 14:02:39 The font is already set on the body.
127 left: 30px;
128 line-height: 24px;
129 position: absolute;
130 top: 30px;
131 }
132
133 .apps-promo-learn-more {
134 background-color: rgb(77, 144, 254);
135 border: 1px solid rgb(47, 91, 183);
136 border-radius: 2px;
137 box-shadow: rgba(0, 0, 0, 0.0980392) 0 1px 1px 0;
arv (Not doing code reviews) 2013/03/25 15:04:38 in case 0.0980391 is too dark?
MAD 2013/03/25 15:32:24 I copied this from the internal style recommendati
arv (Not doing code reviews) 2013/03/26 14:02:39 I just thought it really strange that they it is n
138 color: white;
139 cursor: default;
140 font: normal bold 11px Arial, Helvetica, sans-serif;
141 height: 27px;
142 left: 30px;
143 line-height: 27px;
144 margin: 0;
arv (Not doing code reviews) 2013/03/25 15:04:38 why is this needed?
MAD 2013/03/25 15:32:24 Again, copied from internal styling... I don't min
arv (Not doing code reviews) 2013/03/26 14:02:39 The goal is to not have anything that is not neede
145 outline-color: white;
arv (Not doing code reviews) 2013/03/25 15:04:38 Can't you do "outline: 0 none white"? Which makes
MAD 2013/03/25 15:32:24 Done.
146 outline-style: none;
147 outline-width: 0;
148 padding: 0 8px;
149 position: absolute;
150 text-align: center;
151 text-decoration: none;
152 top: 70px;
153 vertical-align: baseline;
arv (Not doing code reviews) 2013/03/25 15:04:38 baseline is the default, isn't it? I don't unders
MAD 2013/03/25 15:32:24 Done.
154 width: 90px;
155 }
156
157 .apps-promo-learn-more:hover {
158 background-image: -webkit-linear-gradient(
159 top, rgb(77, 144, 254), rgb(53, 122, 232));
160 border: 1px solid rgb(47, 91, 183);
161 }
162
163 .app-launcher-promo > img {
164 bottom: 0;
165 position: absolute;
166 right: 30px;
167 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698