Copy and paste keyboard shortcuts not disabled while a dialog box is shown |
||||||||
Issue descriptionChrome Version: ToT Steps To Reproduce: (1) Open a writable directory in the Files app. (2) Select a file. (3) Hit [Delete] key. See the delete confirmation dialog box appears. (4) Hit [Ctrl]+C then [Ctrl]+V keys in the dialog box. Expected Result: Nothing happens. Actual Result: A copy of the file is created. How frequently does this problem reproduce? (Always, sometimes, hard to reproduce?) Always What is the impact to the user, and is there a workaround? If so, what is it? This may cause unintentional modification of files.
,
Feb 19 2018
,
Feb 28 2018
,
Apr 12 2018
,
Jun 13 2018
While you're looking at dialogs, this one is related :) In general, no keyboard shortcuts should work/the app should not be interactable while a dialog is open.
,
Nov 7
,
Nov 21
,
Nov 21
It would appear we don't eat the commands to copy, cut and paste when the dialog is shown, so it falls through to the default system copy and paste which, wouldn't you know it, works.
,
Nov 22
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/92ad5f16acef530a0d19e7f0f42c53306e73e1ed commit 92ad5f16acef530a0d19e7f0f42c53306e73e1ed Author: Stuart Langley <slangley@google.com> Date: Thu Nov 22 05:37:25 2018 Prevent copy, cut and paste when a dialog is displayed. Bug: 731021 Change-Id: I3a894b0c8e225ed031537a908cdc5a914293b4b7 Reviewed-on: https://chromium-review.googlesource.com/c/1345679 Commit-Queue: Stuart Langley <slangley@chromium.org> Reviewed-by: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#610312} [modify] https://crrev.com/92ad5f16acef530a0d19e7f0f42c53306e73e1ed/chrome/browser/chromeos/file_manager/file_manager_browsertest.cc [modify] https://crrev.com/92ad5f16acef530a0d19e7f0f42c53306e73e1ed/ui/file_manager/file_manager/foreground/js/file_transfer_controller.js [modify] https://crrev.com/92ad5f16acef530a0d19e7f0f42c53306e73e1ed/ui/file_manager/integration_tests/file_manager/keyboard_operations.js
,
Nov 22
|
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by yamaguchi@chromium.org
, Jun 8 2017