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

Unified Diff: ui/app_list/views/search_result_tile_item_view.cc

Issue 2945343003: cros: update app list item view for all apps grid view (Closed)
Patch Set: feedback Created 3 years, 6 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
« no previous file with comments | « ui/app_list/views/app_list_item_view.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/app_list/views/search_result_tile_item_view.cc
diff --git a/ui/app_list/views/search_result_tile_item_view.cc b/ui/app_list/views/search_result_tile_item_view.cc
index 6df3a3b0d87ca28e322f6877a570fb86d92f917b..b5705e4856b720630acbc62b088890f6114d0283 100644
--- a/ui/app_list/views/search_result_tile_item_view.cc
+++ b/ui/app_list/views/search_result_tile_item_view.cc
@@ -18,19 +18,11 @@ namespace app_list {
namespace {
-constexpr int kRecommendationTileWidth = 96;
-constexpr int kRecommendationTileHeight = 99;
-constexpr int kRecommendationIconTopPadding = 24;
-constexpr int kRecommendationTitleSpacing = 10;
-constexpr int kRecommendationTileMaxWidth = 80;
-
constexpr int kSearchTileWidth = 80;
constexpr int kSearchTileHeight = 92;
constexpr int kSearchTileTopPadding = 4;
constexpr int kSearchTitleSpacing = 6;
-constexpr SkColor kRecommendationTileColor = SK_ColorWHITE;
-
constexpr SkColor kSearchTitleColor =
SkColorSetARGBMacro(0xDF, 0x00, 0x00, 0x00);
constexpr SkColor kSearchAppRatingColor =
@@ -108,7 +100,7 @@ void SearchResultTileItemView::SetSearchResult(SearchResult* item) {
// Customize title UI
if (item_->display_type() == SearchResult::DISPLAY_RECOMMENDATION) {
title()->SetFontList(base_font.DeriveWithSizeDelta(1));
- title()->SetEnabledColor(kRecommendationTileColor);
+ title()->SetEnabledColor(kGridTitleColorFullscreen);
} else if (item_->display_type() == SearchResult::DISPLAY_TILE) {
title()->SetFontList(base_font.DeriveWithSizeDelta(1));
title()->SetEnabledColor(kSearchTitleColor);
@@ -221,12 +213,13 @@ void SearchResultTileItemView::Layout() {
}
if (item_->display_type() == SearchResult::DISPLAY_RECOMMENDATION) {
- rect.Inset(0, kRecommendationIconTopPadding, 0, 0);
+ rect.Inset(0, kGridIconTopPadding, 0, 0);
icon()->SetBoundsRect(rect);
- rect.Inset(0, kGridIconDimension + kRecommendationTitleSpacing, 0, 0);
+ rect.Inset(kGridTitleHorizontalPadding,
+ kGridIconDimension + kGridTitleSpacing,
+ kGridTitleHorizontalPadding, 0);
rect.set_height(title()->GetPreferredSize().height());
- rect.set_width(kRecommendationTileMaxWidth);
title()->SetBoundsRect(rect);
} else if (item_->display_type() == SearchResult::DISPLAY_TILE) {
rect.Inset(0, kSearchTileTopPadding, 0, 0);
@@ -266,7 +259,7 @@ void SearchResultTileItemView::Layout() {
gfx::Size SearchResultTileItemView::CalculatePreferredSize() const {
if (is_fullscreen_app_list_enabled_ && item_) {
if (item_->display_type() == SearchResult::DISPLAY_RECOMMENDATION)
- return gfx::Size(kRecommendationTileWidth, kRecommendationTileHeight);
+ return gfx::Size(kGridTileWidth, kGridTileHeight);
if (item_->display_type() == SearchResult::DISPLAY_TILE)
return gfx::Size(kSearchTileWidth, kSearchTileHeight);
}
« no previous file with comments | « ui/app_list/views/app_list_item_view.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698