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

Toolbar not being enabled on device 2, when bookmarks are synced into toolbar #2136

Closed
GeetaSarvadnya opened this issue Nov 14, 2018 · 3 comments · Fixed by brave/brave-core#962

Comments

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Nov 14, 2018

Description

Imported bookmarks from Chrome browser into brave (Device 1- Windows 10). Bookmarks are imported succssfully on Win10 and toolbar bookmarks and folders are displayed properly in win10. After sync with device 2 (Windows 8), toolbar bookmarks and folders are not shown in Toolbar section on Device2.

Devices

Device 1 - Windows 10 (sync creator)
Device 2- Windows 8

Steps to Reproduce

  1. Launch brave beta with --enable-brave-sync flag
  2. Create sync chain on Device 1
  3. Make sure sync is established between two device
  4. Import bookmarks from Chrome browser on Device 1
  5. Wait untill bookmarks synced on Device 2 (Based on amount of BM's)
  6. Open chrome://bookmarks/ and make sure all the bookmarks are synced in Device 2
  7. Observe that the bookmarks and folders are not shown in toolbar section on Device 2

Actual result:

Device 1 - Windows 10

image

Device 2 - Windows 8

Toolbar section is empty

image

Expected result:

After sync toolbar bookmarks and folders should be shown in toolbar section on device2

Reproduces how often:

Always

Brave version (brave://version info)

Brave 0.57.6 Chromium: 71.0.3578.31 (Official Build) beta (64-bit)
Revision c88fdf2a4ce19a713615ca4fbde7a0d0b5fe2363-refs/branch-heads/3578@{#427}
OS Windows

Reproducible on current release:

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

Website problems only:

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

Additional Information

@kjozwiak @LaurenWags @srirambv @btlechowski

@GeetaSarvadnya GeetaSarvadnya added this to the 0.57.x - Beta milestone Nov 14, 2018
@GeetaSarvadnya GeetaSarvadnya added feature-request priority/P5 Not scheduled. Don't anticipate work on this any time soon. QA/Yes and removed QA/Yes bug labels Nov 14, 2018
@GeetaSarvadnya GeetaSarvadnya modified the milestones: 0.57.x - Beta, 1.x Backlog Nov 14, 2018
@kjozwiak
Copy link
Member

@GeetaSarvadnya was the bookmark toolbar enabled on the Win 8 VM? Even though sync completed, it won't automatically enable the bookmark toolbar. Users will need to enable the bookmark toolbars themselves unless we restore that setting from the previous installation. If it was enabled, then it's definitely a bug 👍

@GeetaSarvadnya
Copy link
Author

GeetaSarvadnya commented Nov 14, 2018

@kjozwiak Bookmark toolbar was not enabled by default on Win8 (Device 2). I have just tested again by enabling Show bookmarks bar settings on win8. Now i could see toolbar bookmarks and folders synced properly on Win 8.

@GeetaSarvadnya GeetaSarvadnya changed the title After sync toolbar bookmarks and folders are not shown in toolbar section on device2 Toolbar not being enabled on device 2, when bookmarks are synced into toolbar Nov 14, 2018
@darkdh darkdh self-assigned this Nov 27, 2018
@darkdh darkdh modified the milestones: 1.x Backlog, 0.58.x - Dev Nov 28, 2018
@btlechowski
Copy link

btlechowski commented Dec 12, 2018

Verification passed on

Brave 0.58.11 Chromium: 71.0.3578.80 (Official Build) beta (64-bit)
Revision 2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS Windows 7 x64

Second device was Windows 10 x64.

Used Test Plan from brave/brave-core#962. Checked STR from OP.

Verified passed with

Brave 0.58.12 Chromium: 71.0.3578.80 (Official Build) (64-bit)
Revision 2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS Mac OS X

Verification passed on

Brave 0.59.14 Chromium: 72.0.3626.28 (Official Build) beta(64-bit)
Revision 997b1040b63bac324e815797ba52be0cd8f616ed-refs/branch-heads/3626@{#461}
OS Linux

Used Test Plan from brave/brave-core#962

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