Issue metadata
Sign in to add a comment
|
Files app touch: need "paste" menu item in the default gear menu |
||||||||||||||||||||||||
Issue descriptionChrome Version: 65.0.3288.0 We've added the three-dot menu button to make the context menu of selected files more accessible for touch UI, during the selection mode. The gear(three-dots) menu button of the toolbar should also perform as the "context menu of the current folder" while no file is selected. So it should have "paste" menu item just like the context menu opened by clicking on the blank part of the file list. Currently users need to workaround by two-finger tap on a blank part of the file list.
,
Dec 21 2017
,
Dec 21 2017
Reopening because Issue 796813 is about a different way to open context menu.
,
Dec 21 2017
,
Dec 22 2017
,
Dec 25 2017
> Isn't it nicer to open the context menu on the top, when long-pressing an empty area of the folder? I've filed issue 796813 This looks reasonable as the backward compatibility without affecting the new touch UI, so we'll do this. However, I don't think that way is always accessible in all folders. When there are more files than fit in the window (i.e. having a scroll bar) in the list view mode, such "blank part" does not exist. So I don't think we solely depend on it but need another access way like this Issue. Here's a pending change to add paste item to the overflow menu. It is pending because the UI design is not confirmed. https://chromium-review.googlesource.com/c/chromium/src/+/841962 It will look like the attached screenshot. - Is this overflow menu the right place to add "paste" command? - If so, should we change the ordering of the menu list and add separator? One of the alternative ideas is having "Paste" the first item in the menu and consist an independent group.
,
Jan 24 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/77e995d28edc6fb303ceb1d596fdf17b046c3ef6 commit 77e995d28edc6fb303ceb1d596fdf17b046c3ef6 Author: Tatsuhisa Yamaguchi <yamaguchi@chromium.org> Date: Wed Jan 24 09:09:55 2018 Add paste menu item to the gear menu. Test: browser_tests --gtest_filter=GearMenu* Bug: 796815 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Iab214e1e4e991518e8c8c02a37ebd7a5c93e93f3 Reviewed-on: https://chromium-review.googlesource.com/841962 Commit-Queue: Tatsuhisa Yamaguchi <yamaguchi@chromium.org> Reviewed-by: Naoki Fukino <fukino@chromium.org> Cr-Commit-Position: refs/heads/master@{#531484} [modify] https://crrev.com/77e995d28edc6fb303ceb1d596fdf17b046c3ef6/chrome/browser/chromeos/file_manager/file_manager_browsertest.cc [modify] https://crrev.com/77e995d28edc6fb303ceb1d596fdf17b046c3ef6/ui/file_manager/file_manager/foreground/js/file_manager_commands.js [modify] https://crrev.com/77e995d28edc6fb303ceb1d596fdf17b046c3ef6/ui/file_manager/file_manager/main.html [modify] https://crrev.com/77e995d28edc6fb303ceb1d596fdf17b046c3ef6/ui/file_manager/integration_tests/file_manager/gear_menu.js
,
Jan 30 2018
yamaguchi@ - Just checking if this fix is planned for M65? Looks like even though we resolved the approach over email, I forgot to update the bug owner properly.
,
Jan 30 2018
,
Jan 30 2018
I would like to merge this to M-65 as it's been planned to fix by M65.
,
Jan 30 2018
We've decided not to have this patched into M65, but will be shipped with M66.
,
Jan 31 2018
,
Feb 23 2018
Google Chrome 66.0.3352.0 (Official Build) dev (64-bit) Platform 10431.0.0 (Official Build) dev-channel eve
,
Nov 13
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by satorux@chromium.org
, Dec 21 2017