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

Bookmarks order changes under same folder doesn't work #3216

Closed
darkdh opened this issue Feb 4, 2019 · 1 comment · Fixed by brave/brave-core#1572
Closed

Bookmarks order changes under same folder doesn't work #3216

darkdh opened this issue Feb 4, 2019 · 1 comment · Fixed by brave/brave-core#1572

Comments

@darkdh
Copy link
Member

darkdh commented Feb 4, 2019

Description

Steps to Reproduce

  1. Setup sync between device A and device B with F1, F2 under toolbar
  2. Waiting for F1 and F2 show on both devices
  3. Swap the positions of F1 and F2 to make it F2, F1 on device A

Actual result:

device B still shows F1, F2

Expected result:

device B should have F2, F1

Reproduces how often:

100%

Brave version (brave://version info)

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds?

Website problems only:

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

Additional Information

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Mar 19, 2019

Verification passed on

Brave 0.62.28 Chromium: 73.0.3683.75 (Official Build) beta (64-bit)
Revision 909ee014fcea6828f9a610e6716145bc0b3ebf4a-refs/branch-heads/3683@{#803}
OS Windows 10 OS Build 17134.523

Device1:

image

Device2:
image

Verified passed with

Brave 0.62.31 Chromium: 73.0.3683.75 (Official Build) beta(64-bit)
Revision 909ee014fcea6828f9a610e6716145bc0b3ebf4a-refs/branch-heads/3683@{#803}
OS Mac OS X
  • Verified STR above when syncing with both a VM and mobile.

Verification passed on

Brave 0.62.37 Chromium: 73.0.3683.86 (Official Build) beta (64-bit)
Revision f9b0bec6063ea50ce2b71f5b9abbae7beee319a6-refs/branch-heads/3683@{#858}
OS Linux

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment