NTP5: Making Apps page taller.

chrome/browser/resources/ntp_search/new_tab.css

 /* Copyright (c) 2012 The Chromium Authors. All rights reserved.
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file. */
 
 /* -----------------------------------------------------------------------------
    General Styles
 ----------------------------------------------------------------------------- */
 
 html {
   /* It's necessary to put this here instead of in body in order to get the
@@ skipping 17 matching lines @@
   padding: 0;
 }
 
 /* [hidden] does display:none, but its priority is too low in some cases. */
 [hidden] {
   display: none !important;
 }
 
 #notification-container {
   -webkit-transition: opacity 200ms;
-  position: absolute;
+  height: 18px;
+  padding: 10px 0;
 }
 
 #notification-container.inactive {

[Dan Beam, 2012/12/14 05:15:14]

why not just $('notification-container').hidden from JS?

[pedro (no code reviews), 2012/12/14 05:49:58]

Done.

   -webkit-transition: opacity 200ms;

[Dan Beam, 2012/12/14 05:15:14]

nit: why is this necessary?  is this when .inactive is removed that this
transition applies now?  I don't think transitions work from display: none;
right now.

[pedro (no code reviews), 2012/12/14 05:49:58]

You're right, this is not necessary anymore. We might revisit the animation for
closing a notification later.

+  display: none;
   opacity: 0;

[Dan Beam, 2012/12/14 05:15:14]

nit: why opacity: 0; if display: none;?

[pedro (no code reviews), 2012/12/14 05:49:58]

This CSS rule has been removed.

 }
 
 #notification {
   display: inline-block;
   font-weight: bold;
   white-space: nowrap;
 }
 
 #notification > div > div,
 #notification > div {
@@ skipping 42 matching lines @@
 #bottom-panel-footer,
 #bottom-panel-toolbar {
   margin: 0 auto;
   text-align: center;
 }
 
 #bottom-panel-header {
   padding: 10px 0 20px;
 }
 
-#bottom-panel-toolbar {
-  height: 18px;
-  padding: 10px 0;
-}
-
 #bookmark-bar-spacer {
   height: 48px;
 }
 
 #promo-bubble-anchor {
   bottom: 50px;
   height: 10px;
   left: 0;
   position: absolute;
   width: 90px;
@@ skipping 27 matching lines @@
   text-align: right;
 }
 
 #attribution > span {
   display: block;
 }
 
 .starting-up * {
   -webkit-transition: none !important;
 }