Brave-ios: Reorder causes folders to show up in a weird format on each device

Created on 6 Feb 2019  路  3Comments  路  Source: brave/brave-ios

Description:

There is a weird bug here when bookmarks/folders are reordered on one device
Device 1: iPhone 7+ sync chain creator with existing bookmarks
Device 2: iPad Pro created empty bookmark folder post sync
Reorder all bookmark folders to the end post sync causes the following

Device 1:
image from ios
Device 2:
file 5

Closing the bookmark tray and reopening it shows correctly. Also some times it messes up the ordering and folders start showing up with weird colours

Expected result:
Order should be same on both devices

Reproduces how often: [Easily reproduced, Intermittent Issue]
Easy

Additional information
https://github.com/brave/brave-ios/issues/772

QA Pass - iPhone QYes bug release-noteexclude

Most helpful comment

@GeetaSarvadnya The folder color goes away after closing and opening bookmark panel again, right?
If so, the color bug is related to https://github.com/brave/brave-ios/issues/986

All 3 comments

Verification passed on iPhone 8 with 1.9 (19.03.25.21). I have tried folders re-order between iPhone and Linux. @srirambv - In iPhone after bookmarks folder re-order color of folder changes is that expected?

iPhone:
Bookmarks folders reorder

Linux:
image

Verified passed with 1.9 (19.03.25.21) on iPad 5th Gen - reordering occurred as expected. I did encounter bookmark folder colors (noted below in https://github.com/brave/brave-ios/issues/860#issuecomment-477118376)

Verified passed on 1.9 (19.03.25.21) on iPhone SE

  • Verification passed on iPhone 6/iPhone 7+ and iPad Pro all running 1.9(19.3.25.21)

@GeetaSarvadnya The folder color goes away after closing and opening bookmark panel again, right?
If so, the color bug is related to https://github.com/brave/brave-ios/issues/986

Was this page helpful?
0 / 5 - 0 ratings

Related issues

kjozwiak picture kjozwiak  路  6Comments

VimS picture VimS  路  6Comments

srirambv picture srirambv  路  9Comments

oloverm picture oloverm  路  6Comments

kylehickinson picture kylehickinson  路  6Comments