New issue
Advanced search Search tips

Issue 875780 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 22
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression : In App Launcher App Folder name is seen chopped

Project Member Reported by mmanchala@chromium.org, Aug 20

Issue description

Chrome Version: 70.0.3524.2/10983.0.0 dev-channel Daisy,Kip and  Celes
OS: Chrome

What steps will reproduce the problem?
(1)Sign into User -> Open App Launcher ->Now drag any App into another App so that Folder is created -> drag 20-22 apps so that next page in Folder will be created 
(2)Now move that folder near to Search bar in App Launcher -> Open App Folder and observe Folder name is seen chopped
(Please refer Screenshot)

Expected:  Folder name should not be chopped
Actual: Instead Folder name is seen chopped

This is Regression Issue as same is working fine in 70.0.3519.3/10974.0.0 Celes


 
Actual_ChoppedName.jpg
113 KB View Download
Expected_Name.jpg
106 KB View Download
Cc: -weidongg@chromium.org newcomer@chromium.org
Labels: new-style-launcher
Owner: weidongg@chromium.org
Status: Assigned (was: Untriaged)
This is caused by my fix to avoid overlap with search box. I will take this.
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 21

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/fb13532cbc0676524391864025772f3ce5e61443

commit fb13532cbc0676524391864025772f3ce5e61443
Author: Weidong Guo <weidongg@chromium.org>
Date: Tue Aug 21 18:12:54 2018

Fix folder bounds issue

The fix in http://go/cgcl/1171905 causes the folder being clipped in
lower resolution. So we should still fit the folder inside
AppsContainerView, in the meanwhile, fit the folder under search box.

Bug:  875780 
Change-Id: I147a10ebc1b10a770a23478d5fd22b084ce1d1dd
Reviewed-on: https://chromium-review.googlesource.com/1182620
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Weidong Guo <weidongg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#584825}
[modify] https://crrev.com/fb13532cbc0676524391864025772f3ce5e61443/ash/app_list/views/app_list_folder_view.cc

Status: Fixed (was: Assigned)

Sign in to add a comment