[Merge] lp:~rpadovani/webbrowser-app/new-tab-landscape-mode into lp:webbrowser-app
Ugo Riboni
ugo.riboni at canonical.com
Thu Jun 18 15:46:40 UTC 2015
Review: Needs Fixing
Thanks Riccardo for the work on this.
The bookmarks UrlList needs the limit property to be set on it, or it does not show anything. Design suggests using a limit of 10.
The UI Toolkit team is working on a new component called Sections that will allow you to easily implement the design for the switch between the Top Sites and Bookmarks sections.
You can find the work in this branch: lp:~tpeeters/ubuntu-ui-toolkit/100-SectionBar/
I pushed a branch here, based on yours, that uses the new Sections component: lp:~uriboni/webbrowser-app/new-tab-wide
Please base your branch on the use of Sections, since they will land in the UI toolkit soon.
Finally, the design for this feature calls for remembering the state so that we can go back to the last section that was active when creating a new tab after re-opening the browser.
And you should probably investigate the use of conditional layouts from the UITK to implement the switch between layouts.
--
https://code.launchpad.net/~rpadovani/webbrowser-app/new-tab-landscape-mode/+merge/261327
Your team Ubuntu Phablet Team is subscribed to branch lp:webbrowser-app.
More information about the Ubuntu-reviews
mailing list