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

Side by Side Diff: chrome/browser/resources/ntp4/new_tab.css

Issue 12038090: Layout the NTP footer with flexboxes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 11 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 html { 5 html {
6 /* It's necessary to put this here instead of in body in order to get the 6 /* It's necessary to put this here instead of in body in order to get the
7 background-size of 100% to work properly */ 7 background-size of 100% to work properly */
8 height: 100%; 8 height: 100%;
9 overflow: hidden; 9 overflow: hidden;
10 } 10 }
(...skipping 162 matching lines...) Expand 10 before | Expand all | Expand 10 after
173 } 173 }
174 174
175 /* TODO(estade): remove this border hack and replace with a webkit-gradient 175 /* TODO(estade): remove this border hack and replace with a webkit-gradient
176 * border-image on #footer once WebKit supports border-image-slice. 176 * border-image on #footer once WebKit supports border-image-slice.
177 * See https://bugs.webkit.org/show_bug.cgi?id=20127 */ 177 * See https://bugs.webkit.org/show_bug.cgi?id=20127 */
178 #footer-border { 178 #footer-border {
179 height: 1px; 179 height: 1px;
180 } 180 }
181 181
182 #footer-content { 182 #footer-content {
183 -webkit-box-align: center; 183 -webkit-align-items: center;
184 display: -webkit-box; 184 -webkit-flex-direction: row;
Dan Beam 2013/01/31 20:22:11 this seems to be the default value (so probably no
dconnelly 2013/02/04 16:40:45 Done.
185 -webkit-justify-content: space-between;
Dan Beam 2013/01/31 20:22:11 why is -webkit-justify-content needed?
dconnelly 2013/02/01 14:34:14 Re: justify, .invisible, etc: I think explaining t
186 display: -webkit-flex;
185 height: 49px; 187 height: 49px;
186 } 188 }
187 189
188 #footer-content > * { 190 #footer-content > * {
189 margin: 0 9px; 191 margin: 0 9px;
190 } 192 }
191 193
192 #logo-img { 194 #logo-img {
193 display: inline-block; 195 display: inline-block;
194 margin-top: 4px; 196 margin-top: 4px;
(...skipping 183 matching lines...) Expand 10 before | Expand all | Expand 10 after
378 margin: 0; 380 margin: 0;
379 max-width: 150px; 381 max-width: 150px;
380 min-width: 90px; 382 min-width: 90px;
381 outline: none; 383 outline: none;
382 padding: 0; 384 padding: 0;
383 position: absolute; 385 position: absolute;
384 top: 0; 386 top: 0;
385 z-index: 5; 387 z-index: 5;
386 } 388 }
387 389
390 /* Footer buttons. ************************************************************/
391
388 #chrome-web-store-link { 392 #chrome-web-store-link {
393 -webkit-order: 2;
389 -webkit-padding-end: 12px; 394 -webkit-padding-end: 12px;
390 /* Match transition delay of recently closed button. */ 395 /* Match transition delay of recently closed button. */
391 -webkit-transition-delay: 100ms; 396 -webkit-transition-delay: 100ms;
392 color: inherit; 397 color: inherit;
393 cursor: pointer; 398 cursor: pointer;
394 display: inline-block; 399 display: inline-block;
395 margin: 0; 400 margin: 0;
396 text-decoration: none; 401 text-decoration: none;
397 white-space: nowrap; 402 white-space: nowrap;
398 } 403 }
399 404
400 #chrome-web-store-title { 405 #chrome-web-store-title {
401 -webkit-padding-end: 36px; 406 -webkit-padding-end: 36px;
402 -webkit-padding-start: 15px; 407 -webkit-padding-start: 15px;
403 background: url('chrome://theme/IDR_WEBSTORE_ICON_24') right 50% no-repeat; 408 background: url('chrome://theme/IDR_WEBSTORE_ICON_24') right 50% no-repeat;
404 display: inline-block; 409 display: inline-block;
405 line-height: 49px; 410 line-height: 49px;
406 } 411 }
407 412
413 #chrome-web-store-link.invisible {
414 -webkit-order: -1;
415 visibility: hidden;
416 }
417
408 #chrome-web-store-link:hover { 418 #chrome-web-store-link:hover {
409 color: #666; 419 color: #666;
410 } 420 }
411 421
412 html[dir='rtl'] #chrome-web-store-title { 422 html[dir='rtl'] #chrome-web-store-title {
413 background-position-x: left; 423 background-position-x: left;
414 } 424 }
415 425
416 #vertical-separator { 426 #vertical-separator {
417 background-color: rgb(178, 178, 178); 427 background-color: rgb(178, 178, 178);
418 display: none; 428 display: inline-block;
419 height: 20px; 429 height: 20px;
420 margin: 0; 430 margin: 0;
421 vertical-align: middle; 431 vertical-align: middle;
432 visibility: hidden;
422 width: 1px; 433 width: 1px;
423 } 434 }
424 435
425 /* Show the separator only if one of the menus is visible. */ 436 /* Show the separator only if one of the menus is visible. */
426 .footer-menu-button:not([hidden]) ~ #vertical-separator { 437 .footer-menu-button:not(.invisible) ~ #chrome-web-store-link > #vertical-separat or {
Dan Beam 2013/01/31 20:22:11 80 col wrap, > isn't needed
dconnelly 2013/02/04 16:40:45 Done.
427 display: block; 438 visibility: visible;
428 } 439 }
429 440
430 /* In trash mode, hide the menus and web store link. */ 441 /* In trash mode, hide the menus and web store link. */
431 #footer.showing-trash-mode #chrome-web-store-link,
432 #footer.showing-trash-mode .menu-container { 442 #footer.showing-trash-mode .menu-container {
433 -webkit-transition-delay: 0; 443 -webkit-transition-delay: 0;
434 opacity: 0; 444 opacity: 0;
435 visibility: hidden; 445 visibility: hidden;
436 } 446 }
437 447
438 #footer .menu-container { 448 #footer .menu-container {
439 -webkit-box-align: center; 449 -webkit-align-items: center;
450 -webkit-flex-direction: row;
451 -webkit-justify-content: flex-end;
440 /* Put menus in a box so the order can easily be swapped. */ 452 /* Put menus in a box so the order can easily be swapped. */
441 display: -webkit-box; 453 display: -webkit-flex;
442 height: 100%; 454 height: 100%;
443 margin: 0; 455 margin: 0;
444 } 456 }
445 457
458 #recently-closed-menu-button.invisible {
Dan Beam 2013/01/31 20:22:11 why did you bring back this whole .invisible busin
dconnelly 2013/02/01 14:34:14 The problem is that the button group has variable
459 -webkit-order: -1;
460 visibility: hidden;
461 }
462
463 #recently-closed-menu-button:not(.invisible) {
464 -webkit-order: 1;
465 }
466
467 #other-sessions-menu-button.invisible {
Dan Beam 2013/01/31 20:22:11 combine with selector above, i.e. #chrome-web-s
dconnelly 2013/02/04 16:40:45 Done.
468 -webkit-order: -1;
469 visibility: hidden;
470 }
471
472 #other-sessions-menu-button::not(.invisible) {
Dan Beam 2013/01/31 20:22:11 ::not() should be :not() (:: means psuedo-element,
dconnelly 2013/02/04 16:40:45 Done.
473 -webkit-order: 0;
474 }
475
446 .other-sessions-promo-message { 476 .other-sessions-promo-message {
447 display: none; 477 display: none;
448 padding: 0; 478 padding: 0;
449 } 479 }
450 480
451 .other-sessions-promo-message:only-child { 481 .other-sessions-promo-message:only-child {
452 display: block; 482 display: block;
453 } 483 }
454 484
455 .other-sessions-promo-message p { 485 .other-sessions-promo-message p {
456 margin: 0; 486 margin: 0;
457 } 487 }
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/resources/ntp4/new_tab.html » ('j') | chrome/browser/resources/ntp4/new_tab.html » ('J')

Powered by Google App Engine
This is Rietveld 408576698