sync icon indicating copy to clipboard operation
sync copied to clipboard
verified publisher icon brave

sync : bookmark order inside of folders is not being maintained

Open srirambv opened this issue 8 years ago • 6 comments

Did you search for similar issues before submitting this one? Yes

Description: When syncing a large data set, with nested subfolders, it was noticed that the order of bookmarks is not being maintained.

Device (iPhone5, iPhone6s plus, iPad 3, ?): iPhone 6s Plus

Brave Version: laptop 0.17.3 & ios 1.4 (17.06.15.13)

Steps to reproduce: 1. with clean profiles on both devices, configure for sync 2. import a large amount of bookmarks 3. check the order of bookmarks in folders after they sync to iOS

Actual Behavior the order of the bookmarks is unintelligible

Expected Behavior the order should match the order on laptop

Is this an issue with Beta build? yes

Is this an issue in the currently released version? no

Can this issue be consistently reproduced? yes

Extra QA steps: 1. 2. 3.

** Website problems only:**

  • did you check with Brave Shields down?
  • did you check in Puffin/Dolphin/iCab/Firefox (UIWebView-based browsers)?

Screenshot if needed: screen shot 2017-06-15 at 3 00 23 pm

image uploaded from ios

Any related issues:

srirambv avatar Jun 19 '17 15:06 srirambv

@srirambv move back to iOS and retest with latest beta please

alexwykoff avatar Jun 19 '17 19:06 alexwykoff

is this a dupe of https://github.com/brave/sync/issues/55 ?

diracdeltas avatar Jun 19 '17 19:06 diracdeltas

This involved imported bookmarks so it may be more related to https://github.com/brave/sync/issues/112

alexwykoff avatar Jun 21 '17 17:06 alexwykoff

We don't preserve bookmark order yet

diracdeltas avatar Jun 21 '17 18:06 diracdeltas

Just saw this on Android with the following scenario: Laptop is Pyramid 0, Android is Pyramid 1. Pyramids are synced and both pyramids are open. Import 5k bookmarks on Laptop. About half of the bookmarks were synced to Android (and displayed in order). Closing and reopening both pyramids (first 0, then 1), the remainder of the bookmarks appear to be synced, however those that were added are out of order.

Ex. These bookmarks were synced prior to closing/reopening pyramids: screenshot_2017-07-21-15-36-57

These bookmarks were synced after closing/reopening pyramids. screenshot_2017-07-21-14-54-01

LaurenWags avatar Jul 21 '17 19:07 LaurenWags

Note that this may be resolved (in browser 0.19.x and latest sync version) as I found that the automated tests which relied on the order being incorrect are now incorrect themselves, and the order is preserved, at least for just a couple of bookmarks in a folder. See https://github.com/brave/browser-laptop/pull/11643

petemill avatar Oct 23 '17 23:10 petemill