Commit c613e8b5 authored by Vladislav Kaznacheev's avatar Vladislav Kaznacheev Committed by Commit Bot

Wrap long app shortcut name onto the 2nd line

In the current implementation of SearchResultTileItemView the label
is multiline only for kInstalledApp result type. This CL enables
multiline for kArcAppShortcut type as well.

Bug: 900278
Test: manual
Change-Id: I19c96cccf7a09d1808f3b68e354403672274f304
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1639279
Commit-Queue: Vladislav Kaznacheev <kaznacheev@chromium.org>
Commit-Queue: Jenny Zhang <jennyz@chromium.org>
Auto-Submit: Vladislav Kaznacheev <kaznacheev@chromium.org>
Reviewed-by: default avatarJenny Zhang <jennyz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#665324}
parent 2a1c28a0
......@@ -205,7 +205,8 @@ void SearchResultTileItemView::OnResultChanged() {
title_->SetMultiLine(
(result()->display_type() == ash::SearchResultDisplayType::kTile ||
(IsSuggestedAppTile() && !show_in_apps_page_)) &&
result()->result_type() == ash::SearchResultType::kInstalledApp);
(result()->result_type() == ash::SearchResultType::kInstalledApp ||
result()->result_type() == ash::SearchResultType::kArcAppShortcut));
// If the new icon is null, it's being decoded asynchronously. Not updating it
// now to prevent flickering from showing an empty icon while decoding.
......
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