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

Issue 7749029: [ntp4] User interface adjustments to NTP4's bookmarks page. (Closed)

Created:
9 years, 4 months ago by csilv
Modified:
9 years, 3 months ago
Reviewers:
Evan Stade
CC:
chromium-reviews, estade+watch_chromium.org, arv (Not doing code reviews)
Visibility:
Public.

Description

[ntp4] User interface adjustments to NTP4's bookmarks page. * Disable underlining for links * Add more padding between the button and the title. * Discard the button gradient in favor of white. * Move favicons two lower-left. * Display favicons at natural size (TODO: they need to be size constrained) * Other bookmarks folder will be hidden if empty. BUG=92921 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=98659

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -8 lines) Patch
M chrome/browser/resources/ntp4/bookmarks_page.css View 3 chunks +11 lines, -7 lines 0 comments Download
M chrome/browser/ui/webui/ntp/bookmarks_handler.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
csilv
+estade for review
9 years, 4 months ago (2011-08-26 01:26:28 UTC) #1
Evan Stade
9 years, 4 months ago (2011-08-26 01:38:36 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698