Chromium Code Reviews Chromium Code Reviews
Issue 7553020:
Revert 94715 - Revert 94714 - Improve layout for bookmark-apps. Add treatment for small icons. (Closed)
Base URL: http://git.chromium.org/git/chromium.git@trunk| OLD | NEW |
|---|---|
| 1 /* Copyright (c) 2011 The Chromium Authors. All rights reserved. | 1 /* Copyright (c) 2011 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 | 5 |
| 6 .app { | 6 .app { |
| 7 position: absolute; | 7 position: absolute; |
| 8 text-align: center; | 8 text-align: center; |
| 9 } | 9 } |
| 10 | 10 |
| 11 .app-contents > span { | 11 .app-contents > span { |
| 12 display: block; | 12 display: block; |
| 13 overflow: hidden; | 13 overflow: hidden; |
| 14 text-decoration: none; | 14 text-decoration: none; |
| 15 text-overflow: ellipsis; | 15 text-overflow: ellipsis; |
| 16 white-space: nowrap; | 16 white-space: nowrap; |
| 17 } | 17 } |
| 18 | 18 |
| 19 /* TODO(estade): hover effect? press effect? */ | 19 /* TODO(estade): hover effect? press effect? */ |
| 20 .app-contents > img { | 20 .app-contents > img { |
| 21 cursor: pointer; | 21 cursor: pointer; |
| 22 display: block; | 22 display: block; |
| 23 margin-left: auto; | 23 margin-left: auto; |
| 24 margin-right: auto; | 24 margin-right: auto; |
| 25 /* -webkit-mask-image set by JavaScript to the image source */ | 25 /* -webkit-mask-image set by JavaScript to the image source */ |
| 26 -webkit-mask-size: 100% 100%; | 26 -webkit-mask-size: 100% 100%; |
| 27 } | 27 } |
| 28 | 28 |
| 29 .app-icon-div { | |
| 30 /* TODO(gbillock): find a better style for this div. Better border color? */ | |
| 31 border: 1px solid gray; | |
| 32 border-radius: 4px; | |
| 33 cursor: pointer; | |
| 34 display: -webkit-box; | |
| 35 /* 96px is the suggested icon size for Chrome apps */ | |
|
Evan Stade
2011/08/03 22:17:36
this is almost correct: it's the minimum size we s
Greg Billock
2011/08/03 22:30:13
The suggestion is "make it 128x128, but only use t
| |
| 36 height: 96px; | |
| 37 margin-bottom: 15px; | |
| 38 margin-left: auto; | |
| 39 margin-right: auto; | |
| 40 margin-top: 15px; | |
| 41 vertical-align: middle; | |
| 42 -webkit-box-align: center; | |
| 43 -webkit-box-pack: center; | |
|
Evan Stade
2011/08/03 22:17:36
nit: -webkit-foo goes at the very end
Greg Billock
2011/08/03 22:30:13
Done.
| |
| 44 width: 96px; | |
| 45 } | |
| 46 | |
| 29 .app-context-menu > button:first-child { | 47 .app-context-menu > button:first-child { |
| 30 font-weight: bold; | 48 font-weight: bold; |
| 31 } | 49 } |
| 32 | 50 |
| 33 .app-context-menu { | 51 .app-context-menu { |
| 34 z-index: 1000; | 52 z-index: 1000; |
| 35 } | 53 } |
| 36 | 54 |
| 37 /* Promo */ | 55 /* Promo */ |
| 38 /* Show the promo if the webstore has a promo and is the only app on the page. | 56 /* Show the promo if the webstore has a promo and is the only app on the page. |
| (...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 71 white-space: nowrap; | 89 white-space: nowrap; |
| 72 -webkit-border-image: url('../ntp/g-button-chocobo.png') 6 10 12 6; | 90 -webkit-border-image: url('../ntp/g-button-chocobo.png') 6 10 12 6; |
| 73 } | 91 } |
| 74 | 92 |
| 75 .apps-promo-hide { | 93 .apps-promo-hide { |
| 76 font-size: 90%; | 94 font-size: 90%; |
| 77 margin-top: 2px; | 95 margin-top: 2px; |
| 78 text-decoration: underline; | 96 text-decoration: underline; |
| 79 -webkit-margin-start: 5px; | 97 -webkit-margin-start: 5px; |
| 80 } | 98 } |
| OLD | NEW |
