Code review comment for lp:~bratsche/indicator-session/users-service

Revision history for this message
Matthew Paul Thomas (mpt) wrote :

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Ted Gould wrote on 24/09/09 14:59:
> On Thu, 2009-09-24 at 04:37 +0000, Cody Russell wrote:
>...
>> Should the menuitems even be there if you can never click them? I
>> went ahead and made an API on UsersServiceDbus that can check if
>> the seat can activate sessions, and we can either make the menuitems
>> insensitive or not add them or whatever. For now I didn't change the
>> menu, just added the API.
>
> My thought is let's just hide them. The reason being that then sys
> admins can use this as a lockdown mode. We have a bug on this from OEM
> as well. But we need to hide all the switching items.
>
> CC'ing MPT as well to get his comments.

What menu items are you talking about?

- --
Matthew Paul Thomas
http://mpt.net.nz/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkq7fPAACgkQ6PUxNfU6ecrSFACfT4ei1ceDUHLVHTXLZL3RRG47
tKoAoIUnp8iDhrj2GuDM8fMgH+5+dm4q
=9HY0
-----END PGP SIGNATURE-----

« Back to merge proposal