-
Notifications
You must be signed in to change notification settings - Fork 2.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove unnecessary "edit" dialog after adding a bookmark #35594
Comments
A temporary work-around is to paste I say "temporary" because these settings are removed sometimes. |
That workaround works for now. Hopefully, this can get addressed quickly - Edge still has this functionality. |
Unfortunately temporary didn't last very long, this flag is gone in the latest nightly build. |
Any new workaround(s) since the flag is now gone from normal build, or is this more Chromium "QoL updates" that just regresses everything like normal? |
I just received the latest update and that option is now gone unfortunately! Very annoying, Brave should address this because I bookmark A LOT and the extra click to edit it is very annoying. |
Really hope this could be fixed. In the meantime, Workaround 1: use a bookmark extension instead, like Bookmark Plus. Workaround 2: downgrade and hold at v1.62.165.
|
That extension works really well and you can use any shortcut you want. |
@bsclifton and @rebron Some have opened a Chromium ticket at https://issues.chromium.org/issues/41496433 but from what I'm seeing, this was an upstream change and they intentionally moved flag out of the browser. Curious to see if they see this as a wontfix. There are a moderate amount of people complaining on this change. But guess big question is, what's the priority for this compared to everything else? Would I be correct in thinking this is pretty much something that needs to be resolved upstream and unlikely to get much attention here downstream with Brave? |
@Saoiray Thanks for the pointer to the chromium issue. My preference is the previous UI is well. We'll need to take a look to see how much effort it is to maintain the old dialog. |
Per @mkarolin, it sounds like we can "auto press" the Edit button here and get us into the next/proper dialog. |
I think we should go non-simplified path (launch edit dialog directly) always. |
Awesome. I'm sure a lot of people will be happy if this can be implemented. Thanks for looking into it. |
The above requires |
Verification PASSED on
Reproduced the original issue using the STR/Cases outlined via #35594 (comment) using Using the STR/Cases mentioned via #35594 (comment), ensured that the new behaviour didn't display two modals as per the following: |
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
andDone
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
The text was updated successfully, but these errors were encountered: