Merge lp://qastaging/~ken-vandine/ubuntu-system-settings/consistency_tweaks into lp://qastaging/ubuntu-system-settings

Proposed by Ken VanDine
Status: Merged
Approved by: Sebastien Bacher
Approved revision: 42
Merged at revision: 40
Proposed branch: lp://qastaging/~ken-vandine/ubuntu-system-settings/consistency_tweaks
Merge into: lp://qastaging/ubuntu-system-settings
Diff against target: 83 lines (+20/-29)
3 files modified
plugins/flight-mode/EntryComponent.qml (+7/-23)
src/qml/CategoryGrid.qml (+5/-2)
src/qml/EntryComponent.qml (+8/-4)
To merge this branch: bzr merge lp://qastaging/~ken-vandine/ubuntu-system-settings/consistency_tweaks
Reviewer Review Type Date Requested Status
Sebastien Bacher (community) Approve
Ken VanDine Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+168968@code.qastaging.launchpad.net

Commit message

* Wrap the icons with an UbuntuShape, this makes it look more consistent with
  the shell icons.
* Added a leftMargin in the CategoryGrid so the section titles aren't right
  against the edge
* Make flight-mode a ListItem.Standard to be more consistent with the about
  plugin.

Description of the change

* Wrap the icons with an UbuntuShape, this makes it look more consistent with
  the shell icons.
* Added a leftMargin in the CategoryGrid so the section titles aren't right
  against the edge
* Make flight-mode a ListItem.Standard to be more consistent with the about
  plugin.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Ken VanDine (ken-vandine) wrote :

Looks good

review: Approve
Revision history for this message
Sebastien Bacher (seb128) wrote :

Great, I was just looking at using a standard ListItem there as well, for the record I opened https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1190196 about the control not being centered

The icons look a bit weird in an UbuntuShape but I guess that will fix itself once we have proper icons to use.

I'm also not sure how the system settings code does the icon lookup, I installed the background panel from Laney (lp:~laney/+junk/appearance-panel) and it picks a gnome/16x16 variant of preferences-desktop-wallpaper when we have a 128 svg in humanity...but that's orthogonal to that merge request ;-)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches