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

drag and drop within bookmark folder on bookmarks bar does not work in certain cases #21529

Closed
LaurenWags opened this issue Mar 8, 2022 · 11 comments
Labels
bug Chromium/waiting upstream Issue is in Chromium; we'll likely wait for the fix feature/bookmarks OS/Desktop OS/macOS priority/P5 Not scheduled. Don't anticipate work on this any time soon. QA Pass-macOS QA/Yes release-notes/exclude repros-on-chrome

Comments

@LaurenWags
Copy link
Member

Description

Some users are reporting problems with drag and drop of bookmarks on the bookmarks bar.

See https://community.brave.com/t/impossible-to-reorganize-bookmarks-manually/349427/8 as an example.

I have reproduced this in a few cases below, mainly centered around ordering of bookmarks within a folder from the bookmarks bar. Note, re-ordering folder contents from brave://bookmarks appears to be working as expected.

Reproduces with Chrome 99.0.4844.51.

Steps to Reproduce

Pre-req: Set up some bookmarks + a folder with some contents (or you can import from bookmarks_3_8_22 copy.txt on a clean profile - just rename to .html before importing).

Once you have the above, try the following scenarios

Scenario 1:

  1. Expand the folder on the bookmarks bar
  2. Select a bookmark and attempt to move it to another position in the same folder (ex. move D between A and B if using bookmarks from above)
  3. Bookmark does not move to new position in folder -- white line appears where you want to drop it, but bookmark does not move.

Scenario 2:

  1. Visit a site not bookmarked (ex. rebron.org)
  2. Highlight the full URL
  3. Drag the URL onto the folder and wait for the folder to expand
  4. Attempt to drop the URL in a specific place (ex. between B and C)
  5. Bookmark is not placed into folder, folder remains expanded in view until you click on it to close the folder
    Note - if you just drop the URL onto the folder without waiting for the folder to expand, the URL is added to your folder as a bookmark. It appears as though specifically trying to place the URL in a certain position cannot be achieved this way.

Screen Shot 2022-03-08 at 9 16 08 AM

Scenario 3:

  1. Select a bookmark from the bookmarks bar
  2. Drag the bookmark onto the folder and wait for the folder to expand
  3. Attempt to drop the Bookmark in a specific place (ex. between C and D)
  4. Bookmark is not placed into folder, folder remains expanded in view until you click on it to close the folder
    Note - if you just drop the bookmark onto the folder without waiting for the folder to expand, the bookmark is added to your folder. It appears as though specifically trying to place the bookmark in a certain position cannot be achieved this way.

Screen Shot 2022-03-08 at 9 21 01 AM

Actual result:

Unable to order bookmarks in the cases above

Expected result:

Able to order bookmarks

Reproduces how often:

easily

Brave version (brave://version info)

Brave 1.36.111 Chromium: 99.0.4844.51 (Official Build) (x86_64)
Revision d537ec02474b5afe23684e7963d538896c63ac77-refs/branch-heads/4844@{#875}
OS macOS Version 12.2.1 (Build 21D62)

Version/Channel Information:

  • Can you reproduce this issue with the current release? yes
  • Can you reproduce this issue with the beta channel? yes 1.37.x
  • Can you reproduce this issue with the nightly channel? yes 1.38.x

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields?
  • Does the issue resolve itself when disabling Brave Rewards?
  • Is the issue reproducible on the latest version of Chrome? yes - 99.0.4844.51

Miscellaneous Information:

cc @Brave-Matt @stephendonner

@rebron rebron added Chromium/waiting upstream Issue is in Chromium; we'll likely wait for the fix priority/P5 Not scheduled. Don't anticipate work on this any time soon. OS/macOS labels Mar 8, 2022
@LaurenWags
Copy link
Member Author

@zdenys
Copy link

zdenys commented Mar 10, 2022

The same is happening in Chrome of which there are various reports too:

@mvargasmoran
Copy link

Hello, is there any solution for this?
The issue has some features:

  • While dragging, the bookmark folders, not just dragging to plain (global) bookmarks bar, the folder opens, let's you choose subfolders,
  • On mousepress release, the folder remains open, the bookmark is not added, and the bookmarks folder navigation stays on top even clicking outside
  • To close the opened folders, one needs to go and click over the top folder.

is there a switch, experimental feature or plugin to fix this?

@starvational
Copy link

starvational commented Mar 15, 2022

The same is happening in Chrome of which there are various reports too:

Looks like it's been fixed in Chrome and deployed to their stable channel:
https://bugs.chromium.org/p/chromium/issues/detail?id=1302138

Wondering when Brave will fix this. I'm seeing this issue (Mac OSX) and I'm on the latest version of Brave.
ref: Version 1.36.112 Chromium: 99.0.4844.51 (Official Build) (x86_64)

@Metis77
Copy link

Metis77 commented Mar 16, 2022

having the same issue.

Workaround:
Rename folder and save, without any changes.

@mvargasmoran
Copy link

@Metis77 that didn't worked for me... maybe it's because I have subfolders.

@Metis77
Copy link

Metis77 commented Mar 16, 2022

@mvargasmoran true, that worked only for the root folders.

@LaurenWags
Copy link
Member Author

This issue appears to be fixed for me using the below build. Is anyone else able to confirm?

Brave | 1.36.117 Chromium: 99.0.4844.74 (Official Build) (x86_64)
-- | --
Revision | fee9a47e86e981802390cb0d41c5ed7ea93c4f6f-refs/branch-heads/4844@{#1060}
OS | macOS Version 12.2.1 (Build 21D62)

@mvargasmoran
Copy link

This was Poggers 👍

@stephendonner
Copy link

@LaurenWags I didn't test extensively but LGTM (drop targets actually work where indicated, etc.) 👍

Brave 1.36.117 Chromium: 99.0.4844.74 (Official Build) (x86_64)
Revision fee9a47e86e981802390cb0d41c5ed7ea93c4f6f-refs/branch-heads/4844@{#1060}
OS macOS Version 11.6.3 (Build 20G415)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Chromium/waiting upstream Issue is in Chromium; we'll likely wait for the fix feature/bookmarks OS/Desktop OS/macOS priority/P5 Not scheduled. Don't anticipate work on this any time soon. QA Pass-macOS QA/Yes release-notes/exclude repros-on-chrome
Projects
None yet
Development

No branches or pull requests

8 participants