Commit 7f3afb9f authored by Alex Newcomer's avatar Alex Newcomer Committed by Commit Bot

cros: Fix elided app names

For the new launcher design:
 - Make grid title padding 0.
 - Allow max space for the title.

This brings the app pods up to spec.

Bug: 885338
Change-Id: I41ceb36558c8955079f476bdd8511ccc77e4fb40
Reviewed-on: https://chromium-review.googlesource.com/c/1332415Reviewed-by: default avatarAlex Newcomer <newcomer@chromium.org>
Reviewed-by: default avatarWeidong Guo <weidongg@chromium.org>
Reviewed-by: default avatarMitsuru Oshima <oshima@chromium.org>
Commit-Queue: Alex Newcomer <newcomer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#608027}
parent 1ac52e75
......@@ -555,7 +555,7 @@ void AppListItemView::PaintButtonContents(gfx::Canvas* canvas) {
flags);
}
int preview_circle_radius = GetPreviewCircleRadius();
const int preview_circle_radius = GetPreviewCircleRadius();
if (!preview_circle_radius)
return;
......@@ -841,6 +841,7 @@ void AppListItemView::SetDragUIState() {
SetUIState(UI_STATE_DRAGGING);
}
// static
gfx::Rect AppListItemView::GetIconBoundsForTargetViewBounds(
const gfx::Rect& target_bounds,
const gfx::Size& icon_size) {
......@@ -850,6 +851,7 @@ gfx::Rect AppListItemView::GetIconBoundsForTargetViewBounds(
return rect;
}
// static
gfx::Rect AppListItemView::GetTitleBoundsForTargetViewBounds(
const gfx::Rect& target_bounds,
const gfx::Size& title_size) {
......@@ -861,6 +863,7 @@ gfx::Rect AppListItemView::GetTitleBoundsForTargetViewBounds(
return rect;
}
// static
gfx::Rect AppListItemView::GetProgressBarBoundsForTargetViewBounds(
const gfx::Rect& target_bounds,
const gfx::Size& progress_bar_size) {
......
......@@ -69,7 +69,8 @@ AppListConfig::AppListConfig()
grid_icon_dimension_ = 64;
grid_icon_bottom_padding_ = 24;
grid_title_top_padding_ = 82;
grid_title_width_ = 96;
grid_title_horizontal_padding_ = 0;
grid_title_width_ = grid_tile_width_;
grid_focus_dimension_ = 80;
grid_focus_corner_radius_ = 12;
suggestion_chip_icon_dimension_ = 16;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment