[Merge] lp:~artmello/webbrowser-app/webbrowser-app-bookmarks_top_sites into lp:webbrowser-app

Olivier Tilloy olivier.tilloy at canonical.com
Fri Jun 27 08:37:07 UTC 2014


Can you back out the changes to the bookmarks model? Ugo has been working on a branch to add the timestamp separately (see https://code.launchpad.net/~phablet-team/webbrowser-app/webbrowser-app-bookmark-timestamp/+merge/224117). Additionally, I don’t think we’ll need BookmarksChronologicalModel as we will ensure the original BookmarksModel is always sorted by creation date (AFAIK there is no use case for sorting them otherwise).

So until we merge Ugo’s branch (which needs some rework that I’ll probably take on), bookmarks would not be ordered by creation date in the new tab page, but that shouldn’t be too bad for a start.
-- 
https://code.launchpad.net/~artmello/webbrowser-app/webbrowser-app-bookmarks_top_sites/+merge/224592
Your team Ubuntu Phablet Team is requested to review the proposed merge of lp:~artmello/webbrowser-app/webbrowser-app-bookmarks_top_sites into lp:webbrowser-app.



More information about the Ubuntu-reviews mailing list