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

Issue 692868 link

Starred by 1 user
Status: Fixed
Owner: ----
Closed: Apr 2017
Cc:
calamity@chromium.org
█ tsergeant@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Feature

Blocking:
issue 658980



Sign in to add a comment

[MD Bookmarks] Persist collapsed folders between page loads

Project Member Reported by tsergeant@chromium.org, Feb 16 2017 
Like the non-material bookmark manager, we want to have all folders open by default, but persist collapsed folders in localstorage.

See: https://cs.chromium.org/chromium/src/chrome/browser/resources/bookmark_manager/js/bmm/bookmark_tree.js?type=cs&q=bookmark_tree.js&l=85
Project Member

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

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

commit e9e0139202b4bf30fd7c6bb3baedce981a72e265
Author: calamity <calamity@chromium.org>
Date: Tue Apr 04 05:39:52 2017

[MD Bookmarks] Persist collapsed folders between page loads.

This CL persists the closed folder state to local storage.

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

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

[modify] https://crrev.com/e9e0139202b4bf30fd7c6bb3baedce981a72e265/chrome/browser/browser_resources.grd
[modify] https://crrev.com/e9e0139202b4bf30fd7c6bb3baedce981a72e265/chrome/browser/resources/md_bookmarks/app.html
[modify] https://crrev.com/e9e0139202b4bf30fd7c6bb3baedce981a72e265/chrome/browser/resources/md_bookmarks/app.js
[modify] https://crrev.com/e9e0139202b4bf30fd7c6bb3baedce981a72e265/chrome/browser/resources/md_bookmarks/compiled_resources2.gyp
[add] https://crrev.com/e9e0139202b4bf30fd7c6bb3baedce981a72e265/chrome/browser/resources/md_bookmarks/constants.html
[add] https://crrev.com/e9e0139202b4bf30fd7c6bb3baedce981a72e265/chrome/browser/resources/md_bookmarks/constants.js
[modify] https://crrev.com/e9e0139202b4bf30fd7c6bb3baedce981a72e265/chrome/browser/resources/md_bookmarks/dnd_manager.html
[modify] https://crrev.com/e9e0139202b4bf30fd7c6bb3baedce981a72e265/chrome/browser/resources/md_bookmarks/dnd_manager.js
[modify] https://crrev.com/e9e0139202b4bf30fd7c6bb3baedce981a72e265/chrome/browser/resources/md_bookmarks/reducers.html
[modify] https://crrev.com/e9e0139202b4bf30fd7c6bb3baedce981a72e265/chrome/browser/resources/md_bookmarks/reducers.js
[modify] https://crrev.com/e9e0139202b4bf30fd7c6bb3baedce981a72e265/chrome/browser/resources/md_bookmarks/types.js
[modify] https://crrev.com/e9e0139202b4bf30fd7c6bb3baedce981a72e265/chrome/browser/resources/md_bookmarks/util.js
[modify] https://crrev.com/e9e0139202b4bf30fd7c6bb3baedce981a72e265/chrome/browser/ui/webui/md_bookmarks/md_bookmarks_ui.cc
[add] https://crrev.com/e9e0139202b4bf30fd7c6bb3baedce981a72e265/chrome/test/data/webui/md_bookmarks/app_test.js
[modify] https://crrev.com/e9e0139202b4bf30fd7c6bb3baedce981a72e265/chrome/test/data/webui/md_bookmarks/md_bookmarks_browsertest.js
[modify] https://crrev.com/e9e0139202b4bf30fd7c6bb3baedce981a72e265/chrome/test/data/webui/md_bookmarks/test_store.js

Comment 2 by tsergeant@chromium.org, Apr 21 2017

Status: Fixed (was: Available)
►

Sign in to add a comment