Created attachment 294535 [details] [review]
don't show empty row for remote users

Empty row is shown instead of hidden autologin switch, because autologin box is still visible. Set visibility on autologin box to hide this empty row.

Review of attachment 294535 [details] [review]:

a-c-n

But I wonder: why is the switch contained in a box? A quick search doesn't reveal any use for the box.

(In reply to comment #2)
> Review of attachment 294535 [details] [review]:
> 
> a-c-n

Thanks for the review!
 
> But I wonder: why is the switch contained in a box? A quick search doesn't
> reveal any use for the box.

Don't know why it was there originally. I wanted to remove it, but then the switch is expanded vertically to have same height as other widgets, what is probably wrong, because it should be vertically aligned to center instead. But I don't know how to do that... Would be best to set row-homogeneous to True and remove those size groups, but the icon and name would have to be moved out from the grid.

Comment on attachment 294535 [details] [review]
don't show empty row for remote users

Pushed for master as:
commit b61c74aa607d916eed051a33e98fc53e34fdd50a

and gnome-3-14 as:
commit fa05d4913f512e4411f1ae2945eafb293a662a50