Code review comment for lp://staging/~mardy/signon-ui/qt5

Looks good, but I guess that it should be merged into the 13.04 branch, not 12.10. You might need to change things around to set the development focus in Launchpad, as that is where merge proposals are targetted by default.