Code review comment for lp:~donadigo/wingpanel-indicator-session/switch-to-user

Revision history for this message
Felipe Escoto (philip.scott) wrote :

Alright, so buttons do work! But two things...

Buttons should maybe be non-focusable (Looks very ugly when they're either blue or gray depending on their focus)

there is also a lot of white space around each of the userboxes, if you could get them closer to how the battery or sound plug look it would be awesome!.

Im glad you were able to get switching to other users working :)

review: Needs Fixing

« Back to merge proposal