[Merge] lp:~rpadovani/webbrowser-app/settings-page into lp:webbrowser-app

[Riccardo Padovani]

> In src/app/webbrowser/SettingsPageHeader.qml:
> 
>  - Does the actual header really need to be a ListItem.Empty ? Can’t it just
> be a Rectangle?

Done
 
>  - The 'text' property should be of type string, not var

Done

>  - Can you rename 'trigger' to 'back' ?

Done

>  - Does the ListItem.Divider really need a coloured Rectangle inside itself?
> Isn’t it themed correctly by default?

Yes, it needs the rectangle, otherwise when flickable goes under it it is semitrasparent

>  - The top-level item should be a Column, its anchors shouldn’t be defined in
> the definition of the component itself, and its height should probably be set
> to childrenRect.height.

Done
-- 
https://code.launchpad.net/~rpadovani/webbrowser-app/settings-page/+merge/253975
Your team Ubuntu Phablet Team is subscribed to branch lp:webbrowser-app.