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 6a6d8a96adf8d9acf9a4af8fcfd880f042e3dce8..0b0dd62ef9ba10db29525d1f14369a0e3ddf6b03 100644 |
--- a/ui/app_list/views/apps_grid_view.cc |
+++ b/ui/app_list/views/apps_grid_view.cc |
@@ -60,7 +60,7 @@ namespace { |
const int kDragBufferPx = 20; |
// Padding space in pixels for fixed layout. |
-const int kBottomPadding = 3; |
+const int kBottomPadding = 2; |
const int kLeftRightPadding = 24; |
// Padding space in pixels between pages. |
@@ -70,12 +70,13 @@ const int kPagePadding = 40; |
const int kPreferredTileWidth = 88; |
const int kPreferredTileHeight = 98; |
-const int kExperimentalPreferredTileWidth = 90; |
-const int kExperimentalPreferredTileHeight = 90; |
+const int kExperimentalPreferredTileWidth = 100; |
+const int kExperimentalPreferredTileHeight = 100; |
// Padding on each side of a tile. |
-const int kExperimentalTileLeftRightPadding = 15; |
-const int kExperimentalTileTopBottomPadding = 11; |
+const int kExperimentalTileLeftRightPadding = 10; |
+const int kExperimentalTileBottomPadding = 12; |
+const int kExperimentalTileTopPadding = 6; |
// Width in pixels of the area on the sides that triggers a page flip. |
const int kPageFlipZoneSize = 40; |
@@ -111,6 +112,16 @@ gfx::Size GetTileViewSize() { |
: gfx::Size(kPreferredTileWidth, kPreferredTileHeight); |
} |
+// Returns the padding around a tile view. |
+gfx::Insets GetTilePadding() { |
+ if (!switches::IsExperimentalAppListEnabled()) |
+ return gfx::Insets(); |
+ |
+ return gfx::Insets( |
+ -kExperimentalTileLeftRightPadding, -kExperimentalTileTopPadding, |
+ -kExperimentalTileLeftRightPadding, -kExperimentalTileBottomPadding); |
+} |
+ |
// RowMoveAnimationDelegate is used when moving an item into a different row. |
// Before running the animation, the item's layer is re-created and kept in |
// the original position, then the item is moved to just before its target |
@@ -407,7 +418,7 @@ void AppsGridView::SetLayout(int cols, int rows_per_page) { |
if (switches::IsExperimentalAppListEnabled()) { |
SetBorder(views::Border::CreateEmptyBorder( |
- 0, kExperimentalWindowPadding, 0, kExperimentalWindowPadding)); |
+ 0, kExperimentalAppsGridPadding, 0, kExperimentalAppsGridPadding)); |
} else { |
SetBorder(views::Border::CreateEmptyBorder( |
0, kLeftRightPadding, kBottomPadding, kLeftRightPadding)); |
@@ -419,7 +430,7 @@ gfx::Size AppsGridView::GetTotalTileSize() { |
gfx::Size size = GetTileViewSize(); |
if (switches::IsExperimentalAppListEnabled()) { |
size.Enlarge(2 * kExperimentalTileLeftRightPadding, |
- 2 * kExperimentalTileTopBottomPadding); |
+ kExperimentalTileBottomPadding); |
} |
return size; |
} |
@@ -2120,9 +2131,7 @@ AppsGridView::Index AppsGridView::GetNearestTileIndexForPoint( |
gfx::Size AppsGridView::GetTileGridSize() const { |
gfx::Rect bounds = GetExpectedTileBounds(0, 0); |
bounds.Union(GetExpectedTileBounds(rows_per_page_ - 1, cols_ - 1)); |
- if (switches::IsExperimentalAppListEnabled()) |
- bounds.Inset(-kExperimentalTileLeftRightPadding, |
- -kExperimentalTileTopBottomPadding); |
+ bounds.Inset(GetTilePadding()); |
return bounds.size(); |
} |
@@ -2136,7 +2145,7 @@ gfx::Rect AppsGridView::GetExpectedTileBounds(int row, int col) const { |
gfx::Rect tile_bounds(gfx::Point(bounds.x() + col * total_tile_size.width(), |
bounds.y() + row * total_tile_size.height()), |
total_tile_size); |
- tile_bounds.ClampToCenteredSize(GetTileViewSize()); |
+ tile_bounds.Inset(-GetTilePadding()); |
return tile_bounds; |
} |