New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 874157 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

[Navi] change order of bookmarks after sync

Project Member Reported by scottchen@chromium.org, Aug 14

Issue description

Currently bookmarks that are synced will push back the bookmarks added during the onboarding process, and those onboarding bookmarks might go out of view.

If we decide the work involved is worth it, change this behavior such that synced bookmarks are appended after the onboarding bookmarks.
 
Labels: -Pri-1 Pri-2
Changing to P2 since this is not really blocking the onboarding feature, and it's a bit unsure if we want to do this / can get it done before targeted launch, due to bookmark sync team's comment:

according to mamir@:
"
It's not hard to change and I think we can also avoid introducing any extra data communication with the server while bringing local nodes to the front.

Currently there is an old legacy implementation, and we are working on a new architecture.
Should we change both? I would be less confident when touching the old legacy code.
If not, then the two implementations would diverge and we should maintain two different set of test cases for each implementation."

Components: UI>Browser>FirstRun
Components: -UI>Browser>WebUI

Sign in to add a comment