NTP5: Making Apps page taller.

chrome/browser/resources/ntp_search/new_tab.css

Index: chrome/browser/resources/ntp_search/new_tab.css
diff --git a/chrome/browser/resources/ntp_search/new_tab.css b/chrome/browser/resources/ntp_search/new_tab.css
index 557610e9fd0ead8cf32d361003f041327a52abad..f23bac0e3b0054b4ece2379c950b6bbf80ca37af 100644
--- a/chrome/browser/resources/ntp_search/new_tab.css
+++ b/chrome/browser/resources/ntp_search/new_tab.css
@@ -35,11 +35,13 @@ body {
 
 #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.

 }
 
@@ -103,11 +105,6 @@ body {
   padding: 10px 0 20px;
 }
 
-#bottom-panel-toolbar {
-  height: 18px;
-  padding: 10px 0;
-}
-
 #bookmark-bar-spacer {
   height: 48px;
 }