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

Issue 692837 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Not on Chrome
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Feature

Blocked on:
issue 729856
issue 731486

Blocking:
issue 658980



Sign in to add a comment

[MD Bookmarks] Implement/refine bookmark opening behavior

Project Member Reported by tsergeant@chromium.org, Feb 16 2017

Issue description

It should be possible to open multiple items at once:
* By selecting multiple items and then pressing enter (or shift-double clicking!?!)
* By using Open All from a folder context menu

See previous work: https://codereview.chromium.org/2668693007/
 
Project Member

Comment 1 by bugdroid1@chromium.org, May 4 2017

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

commit 3693ae6f5653c20d845aae55fb2b734b46c337a8
Author: calamity <calamity@chromium.org>
Date: Thu May 04 03:12:43 2017

[cr-action-menu] Allow configurable anchors.

This CL adds showAtPosition to cr-action-menu which allows a client to
specify a rect or point to anchor to and which side of the menu to align.

This will be used in MD Bookmarks for right click context menus which
require more flexible alignment.

BUG= 692837 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/3693ae6f5653c20d845aae55fb2b734b46c337a8/chrome/test/data/webui/cr_elements/cr_action_menu_test.js
[modify] https://crrev.com/3693ae6f5653c20d845aae55fb2b734b46c337a8/ui/webui/resources/cr_elements/cr_action_menu/cr_action_menu.js

Project Member

Comment 2 by bugdroid1@chromium.org, May 4 2017

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

commit a32022824f01698ab161130a96c2832a8226bb46
Author: calamity <calamity@chromium.org>
Date: Thu May 04 06:00:02 2017

[MD Bookmarks] Right click on bookmark items open context menu.

This CL makes the context menu for bookmark items appear when an item is
clicked.

BUG= 692837 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/a32022824f01698ab161130a96c2832a8226bb46/chrome/browser/resources/md_bookmarks/item.js
[modify] https://crrev.com/a32022824f01698ab161130a96c2832a8226bb46/chrome/browser/resources/md_bookmarks/list.html
[modify] https://crrev.com/a32022824f01698ab161130a96c2832a8226bb46/chrome/browser/resources/md_bookmarks/list.js
[modify] https://crrev.com/a32022824f01698ab161130a96c2832a8226bb46/chrome/test/data/webui/md_bookmarks/item_test.js

Project Member

Comment 3 by bugdroid1@chromium.org, May 5 2017

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

commit fad224fbb037adf1a6f37633708732f758f39497
Author: tsergeant <tsergeant@chromium.org>
Date: Fri May 05 04:42:09 2017

MD Bookmarks: Add keyboard shortcut support to bookmarks-command-manager

This adds keyboard shortcuts for Edit, Copy and Delete.

BUG= 692837 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/fad224fbb037adf1a6f37633708732f758f39497/chrome/browser/resources/md_bookmarks/command_manager.html
[modify] https://crrev.com/fad224fbb037adf1a6f37633708732f758f39497/chrome/browser/resources/md_bookmarks/command_manager.js
[modify] https://crrev.com/fad224fbb037adf1a6f37633708732f758f39497/chrome/test/data/webui/md_bookmarks/command_manager_test.js

Owner: tsergeant@chromium.org
Status: Started (was: Available)
Project Member

Comment 6 by bugdroid1@chromium.org, May 15 2017

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

commit 2db36268c223640a0ed5cfb98de3ca949a8a9b94
Author: tsergeant <tsergeant@chromium.org>
Date: Mon May 15 02:47:53 2017

MD Bookmarks: Make CommandManager available as a singleton

This makes the command manager globally available, which is necessary
to allow <bookmark-item>s to execute commands.

This CL is split apart from other changes as the diff from reformatting
the entire file is very large.

BUG= 692837 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/2db36268c223640a0ed5cfb98de3ca949a8a9b94/chrome/browser/resources/md_bookmarks/command_manager.js

Project Member

Comment 7 by bugdroid1@chromium.org, May 19 2017

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

commit 6c5ad90a8e6c3d841df6b7fb01c935879aedf64e
Author: tsergeant <tsergeant@chromium.org>
Date: Fri May 19 14:12:34 2017

MD Bookmarks: Add 'Open' command, to open in either the BMM or in new tabs

This command is triggered by double clicking selected items, or by
pressing Enter (non-Mac) or Meta-Down (Mac).

If a single folder is selected, the command will open that folder in the
bookmark manager. Otherwise, it will open all selected bookmarks in
foreground tabs.

BUG= 692837 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/6c5ad90a8e6c3d841df6b7fb01c935879aedf64e/chrome/browser/resources/md_bookmarks/actions.js
[modify] https://crrev.com/6c5ad90a8e6c3d841df6b7fb01c935879aedf64e/chrome/browser/resources/md_bookmarks/command_manager.js
[modify] https://crrev.com/6c5ad90a8e6c3d841df6b7fb01c935879aedf64e/chrome/browser/resources/md_bookmarks/constants.js
[modify] https://crrev.com/6c5ad90a8e6c3d841df6b7fb01c935879aedf64e/chrome/browser/resources/md_bookmarks/item.html
[modify] https://crrev.com/6c5ad90a8e6c3d841df6b7fb01c935879aedf64e/chrome/browser/resources/md_bookmarks/item.js
[modify] https://crrev.com/6c5ad90a8e6c3d841df6b7fb01c935879aedf64e/chrome/test/data/webui/md_bookmarks/actions_test.js
[modify] https://crrev.com/6c5ad90a8e6c3d841df6b7fb01c935879aedf64e/chrome/test/data/webui/md_bookmarks/command_manager_test.js
[modify] https://crrev.com/6c5ad90a8e6c3d841df6b7fb01c935879aedf64e/chrome/test/data/webui/md_bookmarks/item_test.js
[modify] https://crrev.com/6c5ad90a8e6c3d841df6b7fb01c935879aedf64e/chrome/test/data/webui/md_bookmarks/list_test.js
[modify] https://crrev.com/6c5ad90a8e6c3d841df6b7fb01c935879aedf64e/chrome/test/data/webui/md_bookmarks/test_util.js

Still TODO:

- Add back in middle-click behavior
- Add count of bookmarks to menu labels
Blockedon: 729856
Blockedon: 731486
Status: Fixed (was: Started)

Sign in to add a comment