Skip to content
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

Closed
Brave-Matt opened this issue Jan 25, 2024 · 16 comments · Fixed by brave/brave-core#22489
Closed

Remove unnecessary "edit" dialog after adding a bookmark #35594

Brave-Matt opened this issue Jan 25, 2024 · 16 comments · Fixed by brave/brave-core#22489

Comments

@Brave-Matt
Copy link
Collaborator

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.

image

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-Matt
Copy link
Collaborator Author

+1 https://community.brave.com/t/the-new-way-to-select-bookmark-folders-when-bookmarking-a-website/528186

@mpql
Copy link

mpql commented Jan 28, 2024

A temporary work-around is to paste chrome://flags/#simplified-bookmark-save-flow into your address bar and set the highlighted setting, "Simplified Bookmark Save Flow" to Disabled.

I say "temporary" because these settings are removed sometimes.

@Ryan-Porter66
Copy link

That workaround works for now.

Hopefully, this can get addressed quickly - Edge still has this functionality.
A lot of good features have been changed for the worse lately.

@rebron rebron added this to Untriaged backlog in Front End Jan 30, 2024
@JunkiEDM
Copy link

JunkiEDM commented Feb 14, 2024

A temporary work-around is to paste chrome://flags/#simplified-bookmark-save-flow into your address bar and set the highlighted setting, "Simplified Bookmark Save Flow" to Disabled.

Unfortunately temporary didn't last very long, this flag is gone in the latest nightly build.
For the meantime I guess a janky workaround could be to triple-click the bookmark button until a proper fix.

@Ryan-Porter66
Copy link

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?

@HugoPerseval
Copy link

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 downloaded the old update 121.1.63.154, and moved the files of this update to the brave folder, and forbade installing the update through another program.

@JcynR
Copy link

JcynR commented Feb 23, 2024

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.

@zhaoruibing
Copy link

zhaoruibing commented Feb 25, 2024

Really hope this could be fixed. In the meantime,

Workaround 1: use a bookmark extension instead, like Bookmark Plus.
Downside is the shortcut won't be Ctrl+D.

Workaround 2: downgrade and hold at v1.62.165.
WARNING: not using the latest version implies security risks.

  1. Download Release v1.62.165
  2. Uninstall Brave, select keep browser data.
  3. Install Release v1.62.165.
  4. Set flag brave://flags/#simplified-bookmark-save-flow to 'Disabled'.
  5. Disable Brave auto update.
  6. Relaunch Brave.
  7. (Remember to undo the above when Brave fixes this issue.)

@Bidonh
Copy link

Bidonh commented Mar 2, 2024

Really hope this could be fixed. In the meantime,

Workaround 1: use a bookmark extension instead, like Bookmark Plus. Downside is the shortcut won't be Ctrl+D.

That extension works really well and you can use any shortcut you want.
Including Ctrl + D.
Modify it here brave://extensions/shortcuts

@Saoiray
Copy link

Saoiray commented Mar 3, 2024

@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?

@rebron
Copy link
Collaborator

rebron commented Mar 6, 2024

@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.

@rebron
Copy link
Collaborator

rebron commented Mar 6, 2024

Per @mkarolin, it sounds like we can "auto press" the Edit button here and get us into the next/proper dialog.
cc: @simonhong wdyt?

@rebron rebron added the priority/P2 A bad problem. We might uplift this to the next planned release. label Mar 6, 2024
@simonhong
Copy link
Member

Per @mkarolin, it sounds like we can "auto press" the Edit button here and get us into the next/proper dialog. cc: @simonhong wdyt?

I think we should go non-simplified path (launch edit dialog directly) always.
It would be quick. working now.

@simonhong simonhong self-assigned this Mar 7, 2024
@simonhong simonhong moved this from Untriaged backlog to In progress in Front End Mar 7, 2024
simonhong added a commit to brave/brave-core that referenced this issue Mar 7, 2024
@Saoiray
Copy link

Saoiray commented Mar 7, 2024

Awesome. I'm sure a lot of people will be happy if this can be implemented. Thanks for looking into it.

@kjozwiak kjozwiak moved this from In progress to Completed in Front End Mar 9, 2024
@kjozwiak
Copy link
Member

kjozwiak commented Mar 9, 2024

The above requires 1.63.172 or higher for 1.63.x verification 👍

@GeetaSarvadnya
Copy link
Collaborator

GeetaSarvadnya commented Mar 13, 2024

Verification PASSED on

Brave | 1.63.174 Chromium: 122.0.6261.128 (Official Build) (64-bit)
-- | --
Revision | 0d369d4e1dfee7bcea82774079dc8cd658746c98
OS | Windows 10 Version 22H2 (Build 19045.4046)

Reproduced the original issue using the STR/Cases outlined via #35594 (comment) using 1.63.169 Chromium: 122.0.6261.111 as per the following:
bookmarks edit

Using the STR/Cases mentioned via #35594 (comment), ensured that the new behaviour didn't display two modals as per the following:
bookmarks edit1

@GeetaSarvadnya GeetaSarvadnya added QA/In-Progress Indicates that QA is currently in progress for that particular issue QA Pass-Win64 and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Mar 13, 2024
@rebron rebron removed this from Completed in Front End Apr 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.