[TV] Bookmarks list is shown with a different order than Desktop or Android

RESOLVED FIXED in FxOS-S10 (30Oct)

Status

Firefox OS
Sync
P2
normal
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: ferjm, Assigned: seanlee)

Tracking

unspecified
FxOS-S10 (30Oct)
ARM
Gonk (Firefox OS)
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [partner-cherry-pick])

Attachments

(1 attachment)

Comment hidden (empty)
(Reporter)

Updated

3 years ago
Blocks: 824026
Priority: -- → P2
Target Milestone: --- → FxOS-S10 (30Oct)
(Assignee)

Comment 1

3 years ago
The records in a folder will be sorted based on 'children' property's order.
SyncBrowserDB.getBookmark({parentid: XXXXX}) can do the sorting task.
Assignee: nobody → selee
Status: NEW → ASSIGNED
Created attachment 8679940 [details] [review]
[gaia] weilonge:seanlee/TVBrowser/master/Bug1219162 > mozilla-b2g:master
(Assignee)

Updated

3 years ago
Blocks: 1194108, 1194092
Depends on: 1217349
(Assignee)

Comment 3

3 years ago
Comment on attachment 8679940 [details] [review]
[gaia] weilonge:seanlee/TVBrowser/master/Bug1219162 > mozilla-b2g:master

Hey Yifan, Could you help to review my patch? Thank you!
Attachment #8679940 - Flags: review?(yliao)
Comment on attachment 8679940 [details] [review]
[gaia] weilonge:seanlee/TVBrowser/master/Bug1219162 > mozilla-b2g:master

Thanks! A couple nits on Github.
Attachment #8679940 - Flags: review?(yliao) → review+
QA Whiteboard: [COM=Data Sync]
(Assignee)

Comment 5

3 years ago
Thanks a lot for reviewing this. :)

landed on master: https://github.com/mozilla-b2g/gaia/commit/2e6e0c519e2e3f7fcd5c26b7c2445d50a983e2cc
gaia-test: https://treeherder.mozilla.org/#/jobs?repo=gaia&revision=e56ede639d63019eff7ee05e79f321a78bb2d952
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
(Assignee)

Updated

3 years ago
Whiteboard: [partner-cherry-pick]
You need to log in before you can comment on or make changes to this bug.