diff options
author | estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-09-13 02:57:12 +0000 |
---|---|---|
committer | estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-09-13 02:57:12 +0000 |
commit | d70bbed71daef079f54094a7cb99c27aa176503b (patch) | |
tree | 07758208fa8f393cf05f98dcd1821a712462c14f | |
parent | fd88a8846f9024e104e549188b16803a135e1fc5 (diff) | |
download | chromium_src-d70bbed71daef079f54094a7cb99c27aa176503b.zip chromium_src-d70bbed71daef079f54094a7cb99c27aa176503b.tar.gz chromium_src-d70bbed71daef079f54094a7cb99c27aa176503b.tar.bz2 |
ntp4: make footer look snappier
don't show the navigation dots during startup (otherwise the most visited and bookmarks dots show up visibly faster than apps)
also fix even more fallout of the .hidden css rule change (Recently Closed was getting display: none when we just wanted visibility: hidden)
BUG=92989
TEST=manual
Review URL: http://codereview.chromium.org/7862012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100847 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r-- | chrome/browser/resources/ntp4/nav_dot.css | 4 | ||||
-rw-r--r-- | chrome/browser/resources/ntp4/recently_closed.css | 2 | ||||
-rw-r--r-- | chrome/browser/resources/ntp4/recently_closed.js | 7 |
3 files changed, 10 insertions, 3 deletions
diff --git a/chrome/browser/resources/ntp4/nav_dot.css b/chrome/browser/resources/ntp4/nav_dot.css index 3bf3bd6..e5da9ba 100644 --- a/chrome/browser/resources/ntp4/nav_dot.css +++ b/chrome/browser/resources/ntp4/nav_dot.css @@ -18,6 +18,10 @@ -webkit-box-pack: center; } +html.starting-up #dot-list { + display: none; +} + .dot { box-sizing: border-box; cursor: pointer; diff --git a/chrome/browser/resources/ntp4/recently_closed.css b/chrome/browser/resources/ntp4/recently_closed.css index cb4fa75..35cc5a1 100644 --- a/chrome/browser/resources/ntp4/recently_closed.css +++ b/chrome/browser/resources/ntp4/recently_closed.css @@ -20,7 +20,7 @@ } /* Reserve space for the menu button even when it's hidden. */ -#recently-closed-menu-button[hidden], +#recently-closed-menu-button.invisible, #footer.showing-trash-mode #recently-closed-menu-button { visibility: hidden; } diff --git a/chrome/browser/resources/ntp4/recently_closed.js b/chrome/browser/resources/ntp4/recently_closed.js index 042de8d..95c6126 100644 --- a/chrome/browser/resources/ntp4/recently_closed.js +++ b/chrome/browser/resources/ntp4/recently_closed.js @@ -38,7 +38,7 @@ cr.define('ntp4', function() { document.body.appendChild(this.menu); this.needsRebuild_ = true; - this.hidden = true; + this.classList.add('invisible'); this.anchorType = cr.ui.AnchorType.ABOVE; this.invertLeftRight = true; }, @@ -66,7 +66,10 @@ cr.define('ntp4', function() { set dataItems(dataItems) { this.dataItems_ = dataItems; this.needsRebuild_ = true; - this.hidden = dataItems.length == 0; + if (dataItems.length) + this.classList.remove('invisible'); + else + this.classList.add('invisible'); }, /** |