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

Unified Diff: chrome/browser/resources/ntp4/bookmarks_page.js

Issue 7776020: Revert 98859 - [ntp4] More UI refinements. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 years, 4 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/ntp4/bookmarks_page.js
===================================================================
--- chrome/browser/resources/ntp4/bookmarks_page.js (revision 98871)
+++ chrome/browser/resources/ntp4/bookmarks_page.js (working copy)
@@ -59,10 +59,11 @@
var faviconDiv = this.querySelector('.favicon');
var faviconUrl;
if (this.data.url) {
- faviconUrl = 'chrome://favicon/size/16/' + this.data.url;
+ faviconUrl = 'chrome://favicon/size/32/' + this.data.url;
chrome.send('getFaviconDominantColor',
[faviconUrl, id, 'ntp4.setBookmarksFaviconDominantColor']);
} else {
+ // TODO(csilv): We need a large (32px) icon for this URL.
faviconUrl = 'chrome://theme/IDR_BOOKMARK_BAR_FOLDER';
// TODO(csilv): Should we vary this color by platform?
this.stripeColor = '#919191';
@@ -88,11 +89,8 @@
* animate.
*/
setBounds: function(size, x, y) {
- this.style.width = size + 'px';
- this.style.height = heightForWidth(size) + 'px';
-
+ this.style.width = this.style.height = size + 'px';
this.style.left = x + 'px';
- this.style.right = x + 'px';
this.style.top = y + 'px';
},
@@ -175,16 +173,6 @@
TilePage.initGridValues(bookmarksPageGridValues);
/**
- * Calculates the height for a bookmarks tile for a given width. The size
- * is based on a desired size of 96x72 ratio.
- * @return {number} The height.
- */
- function heightForWidth(width) {
- // The 2s are for borders, the 31 is for the title.
- return (width - 2) * 72 / 96 + 2 + 31;
- }
-
- /**
* Creates a new BookmarksPage object.
* @constructor
* @extends {TilePage}
@@ -287,9 +275,6 @@
return false;
},
- /** @inheritDoc */
- heightForWidth: heightForWidth,
-
/**
* Invoked before a batch import begins. We will ignore added/changed
* notifications while the operation is in progress.
« no previous file with comments | « chrome/browser/resources/ntp4/bookmarks_page.css ('k') | chrome/browser/resources/ntp4/most_visited_page.css » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698