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

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

Issue 2949733002: Show Play Store rating and price in app list (Closed)
Patch Set: Apply fix to patch set 5 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
Index: ui/app_list/views/search_result_tile_item_list_view.cc
diff --git a/ui/app_list/views/search_result_tile_item_list_view.cc b/ui/app_list/views/search_result_tile_item_list_view.cc
index 78ff7979072aa3c74a2ec908afd7873b51b6cb71..446d70041c6298ad5d511d61838ac3ecd9143691 100644
--- a/ui/app_list/views/search_result_tile_item_list_view.cc
+++ b/ui/app_list/views/search_result_tile_item_list_view.cc
@@ -8,6 +8,7 @@
#include "base/i18n/rtl.h"
#include "ui/app_list/app_list_constants.h"
+#include "ui/app_list/app_list_features.h"
#include "ui/app_list/app_list_view_delegate.h"
#include "ui/app_list/search_result.h"
#include "ui/app_list/views/search_result_tile_item_view.h"
@@ -20,10 +21,15 @@
namespace {
// Layout constants.
-const size_t kNumSearchResultTiles = 8;
-const int kHorizontalBorderSpacing = 1;
-const int kBetweenTileSpacing = 2;
-const int kTopBottomPadding = 8;
+constexpr size_t kNumSearchResultTiles = 8;
+constexpr int kHorizontalBorderSpacing = 1;
+constexpr int kBetweenTileSpacing = 2;
+constexpr int kTopBottomPadding = 8;
+
+// Layout constants used when fullscreen app list feature is enabled.
+constexpr int kItemListVerticalSpacing = 16;
+constexpr int kItemListHorizontalSpacing = 12;
+constexpr int kBetweenItemSpacing = 8;
} // namespace
@@ -32,17 +38,27 @@ namespace app_list {
SearchResultTileItemListView::SearchResultTileItemListView(
views::Textfield* search_box,
AppListViewDelegate* view_delegate)
- : search_box_(search_box) {
- SetLayoutManager(new views::BoxLayout(
- views::BoxLayout::kHorizontal, gfx::Insets(0, kHorizontalBorderSpacing),
- kBetweenTileSpacing));
+ : search_box_(search_box),
+ is_fullscreen_app_list_enabled_(features::IsFullscreenAppListEnabled()) {
+ if (is_fullscreen_app_list_enabled_) {
+ SetLayoutManager(new views::BoxLayout(
+ views::BoxLayout::kHorizontal,
+ gfx::Insets(kItemListVerticalSpacing, kItemListHorizontalSpacing),
+ kBetweenItemSpacing));
+ } else {
+ SetLayoutManager(new views::BoxLayout(
+ views::BoxLayout::kHorizontal, gfx::Insets(0, kHorizontalBorderSpacing),
+ kBetweenTileSpacing));
+ }
for (size_t i = 0; i < kNumSearchResultTiles; ++i) {
SearchResultTileItemView* tile_item =
new SearchResultTileItemView(this, view_delegate);
tile_item->SetParentBackgroundColor(kCardBackgroundColor);
- tile_item->SetBorder(
- views::CreateEmptyBorder(kTopBottomPadding, 0, kTopBottomPadding, 0));
+ if (!is_fullscreen_app_list_enabled_) {
+ tile_item->SetBorder(
+ views::CreateEmptyBorder(kTopBottomPadding, 0, kTopBottomPadding, 0));
+ }
tile_views_.push_back(tile_item);
AddChildView(tile_item);
}
« no previous file with comments | « ui/app_list/views/search_result_tile_item_list_view.h ('k') | ui/app_list/views/search_result_tile_item_view.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698