New issue
Advanced search Search tips

Issue 883659 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression: Delay with transparent overlay is seen after clicking delete in Files app

Project Member Reported by rkalavakuntla@chromium.org, Sep 13

Issue description

Chrome Version:71.0.3544.0/11061.0.0 dev channel Cyan,Gnawty
OS:Chrome OS

What steps will reproduce the problem?
(1)Sign into user >> Take a screenshot >>Show in folder >>click delete and observe

Actual: Delay with transparent overlay is seen after clicking delete
Expected: No such delay with transparent overlay should  be seen after clicking delete

This is a Regression issue as same is working fine in M-69 beta

Attached the screencast for reference..

 
ACtual transparent overlay.png
104 KB View Download
Actual.mp4
6.8 MB View Download
Expected.mp4
3.1 MB View Download
Labels: -Pri-1 Pri-2
Owner: lucmult@chromium.org
Status: Assigned (was: Untriaged)
Repro'd on 71.0.3551.0

The delay is less than a second, but I guess the expectation is that it's 0 (or reasonably close to).
Status: Started (was: Assigned)
Labels: CrOSFilesCategory-UI
Project Member

Comment 5 by bugdroid1@chromium.org, Oct 2

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

commit 39778d0895e5a1055f4d4f75e394596ea150985f
Author: Luciano Pacheco <lucmult@chromium.org>
Date: Tue Oct 02 02:41:21 2018

Change Dialog background color only when it's visible

This fixes the perception to users that the dialog is taking too long
to disappear.

Test: No change in behavior, so current test suite should cover it. Additionally, manually tested file manager file deletion confirmation.
Bug:  883659 
Change-Id: I414a7282f70a64c0826954ef850a16e0bc37182e
Reviewed-on: https://chromium-review.googlesource.com/1242725
Reviewed-by: Joel Hockey <joelhockey@chromium.org>
Commit-Queue: Luciano Pacheco <lucmult@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595703}
[modify] https://crrev.com/39778d0895e5a1055f4d4f75e394596ea150985f/ui/file_manager/file_manager/foreground/css/common.css

Status: Fixed (was: Started)

Sign in to add a comment