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

Issue 7602017: ntp4: respect non-default apps promo logo (Closed)

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

Description

ntp4: respect non-default apps promo logo BUG=none TEST=manual (set logo to something else in preferences file) Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=96688

Patch Set 1 #

Patch Set 2 : . #

Patch Set 3 : sync'd against crx-less app change #

Unified diffs Side-by-side diffs Delta from patch set Stats (+44 lines, -11 lines) Patch
M chrome/browser/resources/ntp4/apps_page.css View 1 2 2 chunks +22 lines, -2 lines 0 comments Download
M chrome/browser/resources/ntp4/apps_page.js View 1 2 5 chunks +16 lines, -6 lines 0 comments Download
M chrome/browser/resources/ntp4/nav_dot.js View 1 2 1 chunk +3 lines, -1 line 0 comments Download
M chrome/browser/resources/ntp4/tile_page.js View 1 2 2 chunks +3 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Evan Stade
9 years, 4 months ago (2011-08-10 15:45:05 UTC) #1
Rick Byers
LGTM The promo-related CSS is getting pretty complicated - seems like it should be possible ...
9 years, 4 months ago (2011-08-10 17:10:28 UTC) #2
Evan Stade
I too wish I knew how to make it simpler
9 years, 4 months ago (2011-08-10 17:25:07 UTC) #3
commit-bot: I haz the power
9 years, 4 months ago (2011-08-13 05:54:22 UTC) #4
Change committed as 96688

Powered by Google App Engine
This is Rietveld 408576698