BGWhHV
March 1, 2024, 6:47am
1
Remove the recent feature where Brave assumes which folder you wish to put a new bookmark in
6 Likes
it’s a little annoying since you have to triple-click: once to bookmark, once again to edit, and once again to choose the location of where to save. Lastly, clicking the “done” button. I would’ve appreciated if it was like before where it just straight up had the overlay on the bottom showing which folder it was saved to with the dropdown available. Also, removing a bookmark is just as a hassle since there’s no straight up “remove” button on first bookmark even if you click on the bookmark icon itself i.e. you accidentally bookmark, you have to click again on the icon (which is a current workaround for pulling up the folder location quickly). I think Brave can make it cleaner too for those with multiple folders since it can get condensed and the way the popups shape can get messy. Also the huge bookmark icon/image space is a missed opportunity to further edit or make use of that space for interaction: literally a sizeable square with a tiny icon in the middle, what for?
5 Likes
I agree, it’s incredibly annoying and it adds unnecessary writes to SSDs. Please revert to where we could choose which folder to save the bookmark into.
Yeah, agree. Who is coding BM’s is goofy. Jump to top of folder after deleting an item/ wasted spacing/ now this assume folder change. Urrgg.
I found double clicking the BM icon quickly first returns the previous style dialoge box.
artag
March 7, 2024, 8:41pm
5
Another vote for returning to the previous method
The change I would like is to increase the length of the recently-used list of bookmark folders. I find it’s forgotten my common places too quickly. I wish it were of configurable length or, if not comfigurable, about twice as long as latterly.
2 Likes
I’m closing this topic as it’s something of a duplicate (yes, this one came first, but other just happened to get the responses). This is being worked on. You can see similar topic at Bookmark Manager is Too Complicated
And Github links are below:
opened 11:43PM - 25 Jan 24 UTC
feature/user-interface
feature/bookmarks
priority/P2
QA/Yes
release-notes/include
feature-request
OS/Desktop
<!-- Have you searched for similar issues? Before submitting this issue, please … check the open issues and add a note before logging a new issue.
PLEASE USE THE TEMPLATE BELOW TO PROVIDE INFORMATION ABOUT THE ISSUE.
INSUFFICIENT INFO WILL GET THE ISSUE CLOSED. IT WILL ONLY BE REOPENED AFTER SUFFICIENT INFO IS PROVIDED-->
## Description
Part of the Chromium bookmarks "upgrades", there is now an additional dialog box that appears after adding a bookmark, that only says you've added the bookmark and has `Edit` and `Done` options on it.

Clicking `Edit` reveals the edit bookmark drop-down that is what previously appeared after adding a bookmark. This additional dialog doesn't provide any value to us and only requires additional work for the users. I suggest we remove this entirely and revert to the old behavior.
Community thread on the issue:
https://community.brave.com/t/editing-name-and-folder-of-new-bookmarks/528195
brave:master
← brave:disable_simplified_bookmarks_bubble
opened 01:46AM - 07 Mar 24 UTC
fix https://github.com/brave/brave-browser/issues/35594
Resolves
<!-- CI… -related labels that can be applied to this PR:
* CI/run-audit-deps (1) - check for known npm/cargo vulnerabilities (audit_deps)
* CI/run-network-audit (1) - run network-audit
* CI/run-upstream-tests - run Chromium unit and browser tests on Linux and Windows (otherwise only on Linux)
* CI/run-linux-arm64, CI/run-macos-arm64, CI/run-windows-arm64, CI/run-windows-x86 - run builds that would otherwise be skipped
* CI/skip - do not run CI builds (except noplatform)
* CI/skip-linux-x64, CI/skip-android, CI/skip-macos-x64, CI/skip-ios, CI/skip-windows-x64 - skip CI builds for specific platforms
* CI/skip-upstream-tests - do not run Chromium unit, or browser tests (otherwise only on Linux)
* CI/skip-all-linters - do not run presubmit and lint checks
* CI/storybook-url (1) - deploy storybook and provide a unique URL for each build
(1) applied automatically when some files are changed (see: https://github.com/brave/brave-core/blob/master/.github/labeler.yml)
-->
## Submitter Checklist:
- [x] I confirm that no [security/privacy review is needed](https://github.com/brave/brave-browser/wiki/Security-reviews) and no other type of reviews are needed, or that I have [requested](https://github.com/brave/reviews/issues/new/choose) them
- [x] There is a [ticket](https://github.com/brave/brave-browser/issues) for my issue
- [x] Used Github [auto-closing keywords](https://docs.github.com/en/github/managing-your-work-on-github/linking-a-pull-request-to-an-issue) in the PR description above
- [x] Wrote a good [PR/commit description](https://google.github.io/eng-practices/review/developer/cl-descriptions.html)
- [ ] Squashed any review feedback or "fixup" commits before merge, so that history is a record of what happened in the repo, not your PR
- [x] Added appropriate labels (`QA/Yes` or `QA/No`; `release-notes/include` or `release-notes/exclude`; `OS/...`) to the associated issue
- [ ] Checked the PR locally:
* `npm run test -- brave_browser_tests`, `npm run test -- brave_unit_tests` [wiki](https://github.com/brave/brave-browser/wiki/Tests)
* `npm run presubmit` [wiki](https://github.com/brave/brave-browser/wiki/Presubmit-checks), `npm run gn_check`, `npm run tslint`
- [x] Ran `git rebase master` (if needed)
## Reviewer Checklist:
- [ ] A security review [is not needed](https://github.com/brave/brave-browser/wiki/Security-reviews), or a link to one is included in the PR description
- [ ] New files have MPL-2.0 license header
- [ ] Adequate test coverage exists to prevent regressions
- [ ] Major classes, functions and non-trivial code blocks are well-commented
- [ ] Changes in component dependencies are properly reflected in `gn`
- [ ] Code follows the [style guide](https://chromium.googlesource.com/chromium/src/+/HEAD/styleguide/c++/c++.md)
- [ ] Test plan is specified in PR before merging
## After-merge Checklist:
- [ ] The associated issue milestone is set to the smallest version that the
changes has landed on
- [ ] All relevant documentation has been updated, for instance:
- [ ] https://github.com/brave/brave-browser/wiki/Deviations-from-Chromium-(features-we-disable-or-remove)
- [ ] https://github.com/brave/brave-browser/wiki/Proxy-redirected-URLs
- [ ] https://github.com/brave/brave-browser/wiki/Fingerprinting-Protections
- [ ] https://github.com/brave/brave-browser/wiki/Brave%E2%80%99s-Use-of-Referral-Codes
- [ ] https://github.com/brave/brave-browser/wiki/Web-Compatibility-Exceptions-in-Brave
- [ ] https://github.com/brave/brave-browser/wiki/QA-Guide
- [ ] https://github.com/brave/brave-browser/wiki/P3A
## Test Plan:
1. Launch brave and click bookmarks button
2. Check edit bubble is launched always
Fix is incoming.