Issue metadata
Sign in to add a comment
|
Regression: Delay with transparent overlay is seen after clicking delete in Files app |
||||||||||||||||||||||
Issue descriptionChrome 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..
,
Sep 25
This comes from the WebUI Dialog code [1], which delays the hide for 500ms [2]. I'll see what I can do there. [1] - https://cs.chromium.org/chromium/src/ui/webui/resources/js/cr/ui/dialogs.js?l=39&rcl=a3be34098f115e8a6f9a91821de47e5fe8e8163e [2] - https://cs.chromium.org/chromium/src/ui/webui/resources/js/cr/ui/dialogs.js?l=282-291&rcl=a3be34098f115e8a6f9a91821de47e5fe8e8163e
,
Sep 25
,
Sep 27
,
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
,
Oct 2
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by amistry@chromium.org
, Sep 17Owner: lucmult@chromium.org
Status: Assigned (was: Untriaged)