Chromium Code Reviews| OLD | NEW |
|---|---|
| 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 Loading... | |
| 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 } | |
| OLD | NEW |