New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 658745 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

File picker: <esc> should close preview but not dialog

Project Member Reported by tbuck...@chromium.org, Oct 24 2016

Issue description

Chrome Version: 55.0.2883.17
OS Version: 8872.15.0

What steps will reproduce the problem?
(1) Click an "Attach a file" button on crbug.com
(2) Select a screenshot
(3) Press <spacebar> to preview screenshot
(4) Press <esc> to go back to list

What is the expected output?
Preview closes but file selection dialog does not

What do you see instead?
File selection dialog also closes without letting you select a file

Please use labels and text to provide additional information.

 
Cc: oka@chromium.org

Comment 2 by oka@chromium.org, Nov 7 2016

Cc: weifangsun@chromium.org
Owner: oka@chromium.org

Comment 3 by oka@chromium.org, Nov 7 2016

Status: Started (was: Assigned)

Comment 4 by oka@chromium.org, Nov 7 2016

Cc: yamaguchi@chromium.org
Project Member

Comment 5 by bugdroid1@chromium.org, Nov 9 2016

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

commit 4ef31bd8ba19d23fa815ae76dcedaa9b77751d72
Author: oka <oka@chromium.org>
Date: Wed Nov 09 09:15:15 2016

Prevent Escape key from closing open file dialog when Quick View is opening.

TEST=manually.
BUG= 658745 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2480123002
Cr-Commit-Position: refs/heads/master@{#430900}

[modify] https://crrev.com/4ef31bd8ba19d23fa815ae76dcedaa9b77751d72/ui/file_manager/file_manager/foreground/js/quick_view_controller.js

Comment 6 by oka@chromium.org, Nov 11 2016

Status: Fixed (was: Started)
Status: Verified (was: Fixed)
Verified on ChromeOS 8989.0.0, 56.0.2919.0

Sign in to add a comment