Previous status remains selected when presence options are disabled

[Dylan McCall]

Binary package hint: indicator-me

As we know, when Empathy not running, the menu items to set presence are disabled. When the Me menu is first started, none of those menu items are selected, either; it is clear that you simply are not present in those accounts.

However, if I then open Empathy (which enables the menu items) and then quit, (which disables the menu items), the previous status remains selected. It suggests that I am still connected and am unable to change that.

Ideally, the selection should be cleared so that nothing appears selected, but another reasonable option (if the dbus interface can't do that) would be to select the “Offline” menu item so it at least communicates correct information.

ProblemType: Bug
DistroRelease: Ubuntu 10.04
Package: indicator-me 0.2.6-0ubuntu1
ProcVersionSignature: Ubuntu 2.6.32-21.32-generic 2.6.32.11+drm33.2
Uname: Linux 2.6.32-21-generic i686
NonfreeKernelModules: nvidia
Architecture: i386
Date: Sat Apr 17 10:13:16 2010
ProcEnviron:
 LANG=en_CA.utf8
 SHELL=/bin/zsh
SourcePackage: indicator-me

[David Barth (dbarth)]

Design/integration issue to be discussed at UDS.

[David Barth (dbarth)]

The check marks are now correctly cleared when the user is in offline mode (rev 104). This should avoid confusion when looking at the menu items.

[David Barth (dbarth)]

This is a request for a Feature Freeze Exception, per https://wiki.ubuntu.com/FreezeExceptionProcess#General Instructions

Proposed changes: The proposed change fixes a usability bug where the menu was incorrectly reflecting the online status of the user. With the change, the offline status is properly checked when the change happens. The menu items stay inactive however to prevent inconsistency issues that could arise from diverging behavior of the IM clients that can't be all reactivated remotely once they've gone offline.

Rationale for the exception: The change fixes a usability bug
Additional information:
 * the patch is available at http://bazaar.launchpad.net/~indicator-applet-developers/indicator-me/trunk/revision/106
 * the patch does not introduce string changes; the impact on documentation screenshot should be mostly unnoticeable
 * the patch has been reviewed according to the standard code review process for DX developments (using LP code reviews)
 * the patch has been tested by the developer and made available in our internal daily builds for a few days

[David Barth (dbarth)]

Now available as release 0.2.9 of the project

[Omer Akram (om26er)]

this bug is fixed in Maverick.