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

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

Issue 657653002: Move app list search box into AppListView. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 6 years, 2 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/apps_grid_view.cc
diff --git a/ui/app_list/views/apps_grid_view.cc b/ui/app_list/views/apps_grid_view.cc
index 764ba014d14d5233310d9eb97e70ae0f479ecc24..7b3e1c41965852b568010ffe96977ab6cf063b61 100644
--- a/ui/app_list/views/apps_grid_view.cc
+++ b/ui/app_list/views/apps_grid_view.cc
@@ -60,8 +60,7 @@ namespace {
const int kDragBufferPx = 20;
// Padding space in pixels for fixed layout.
-const int kTopPadding = 1;
-const int kBottomPadding = 1;
+const int kBottomPadding = 3;
const int kLeftRightPadding = 24;
// Padding space in pixels between pages.
@@ -426,13 +425,11 @@ void AppsGridView::SetLayout(int cols, int rows_per_page) {
rows_per_page_ = rows_per_page;
if (switches::IsExperimentalAppListEnabled()) {
- SetBorder(views::Border::CreateEmptyBorder(0,
- kExperimentalWindowPadding,
- kBottomPadding,
- kExperimentalWindowPadding));
+ SetBorder(views::Border::CreateEmptyBorder(
+ 0, kExperimentalWindowPadding, 0, kExperimentalWindowPadding));
} else {
SetBorder(views::Border::CreateEmptyBorder(
- kTopPadding, kLeftRightPadding, kBottomPadding, kLeftRightPadding));
+ 0, kLeftRightPadding, kBottomPadding, kLeftRightPadding));
}
}

Powered by Google App Engine
This is Rietveld 408576698