Bookmark cannot be moved to bookmark folder on bookmark toolbar

[luixxiul]

Describe the issue you encountered: Bookmark cannot be moved to bookmark folder on bookmark toolbar.

• Platform (Win7, 8, 10? macOS? Linux distro?): macOS

• Brave Version (revision SHA): 2ceca28

• Steps to reproduce: Test that moving a bookmark into a folder by drag and drop on the bookmarks folder works

• Actual result: it does not work

• Expected result: it should work

• Any related issues:

[tblu]

confirmed on win10, rev 4e46480

[NejcZdovc]

@luixxiul can you please try it on the latest master? For me it's working, but it's hard to nail it 😃

[kiyomasa-slotet]

same issue happened.

• Platform (Win7, 8, 10? macOS? Linux distro?):
  OS X 10.11.6

• Brave Version (revision SHA):
  0.18.14

[luixxiul]

@pullingour I'm not quite sure how this is the same one.

[kiyomasa-slotet]

@luixxiul Do I have to create a new issue?

[luixxiul]

@pullingour yes please, thanks.

[sjlehn]

@pullingour I just entered a defect for that #10323

[bsclifton]

@pullingour In your screenshot, it shows you drag to the .ca folder... Can you try this again, but this time click somewhere else (like inside the page)... then re-click the .ca folder?

I believe the issue is that the context menu didn't get re-rendered yet (with the new item inside). Once you force the app to re-create + show the menu again, it usually shows the correct menu

[kiyomasa-slotet]

@sjlehn Thanks! I'll post description and screenshot for #10323

@bsclifton I tried again, the same problem happened. I'll post another screenshot for #10323
This issue happen when I try drag and drop bookmark that top of folder only. Another bookmarks are fine. I will write more detailed explanation for #10323

[alexwykoff]

@luixxiul this needs comparison vs other browsers. the interaction makes sense for files but we need to compare vs browsers

[luixxiul]

This is the issue I reported:

Actually this is a regression based on 0.18.14, so I think we don't need to do comparisons.

[ghost]

Removing from the milestone, and will be addressed when drag/drop in the browser is addressed with more focus.

[luixxiul]

This seems to have been fixed with 0.18.26. Assigning the milestone for QA