Bug 304343 - disabling and re-enabling an account fast gives unexpected UI behaviour
Summary: disabling and re-enabling an account fast gives unexpected UI behaviour
Status: RESOLVED WORKSFORME
Alias: None
Product: telepathy
Classification: Frameworks and Libraries
Component: accounts-kcm (show other bugs)
Version: git-latest
Platform: unspecified Linux
: NOR normal
Target Milestone: Future
Assignee: Telepathy Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-31 16:05 UTC by George Kiagiadakis
Modified: 2012-10-26 09:32 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description George Kiagiadakis 2012-07-31 16:05:12 UTC 
If you go and disable an account and then re-enable it fast, the UI behaves weird. The account is usually grayed out while it is connecting.

This happens because the model/view architecture does not allow us to wait for the setEnabled() pending operation to be completed and therefore the account is re-enabled before it even gets disabled on MC.
Comment 1 David Edmundson 2012-10-26 04:39:41 UTC 
I can't reproduce this. I've selected an account and pressed space really fast (which disables/enables) and all is absolutely fine. Clicking quickly tends to trigger the account edit dialog

I can see how it's theoretically an issue, but practically I'm not able to reproduce it.

Can anyone else? 
Please repoen if anyone can actually reproduce this.
Comment 2 Martin Klapetek 2012-10-26 08:10:24 UTC 
I used to be able to reproduce it, now I can't anymore, all works as supposed.
Comment 3 George Kiagiadakis 2012-10-26 09:32:37 UTC 
I am not able to reproduce the original issue either. There is still a minor painting issue though: Select a connected account, disable it and re-enable it fast from the check box. The edit account dialog will pop up, press cancel. The checkbox now has a grayed-out tick on it, but it is disabled. You can also reproduce it in the opposite way, with a disabled account, enabling it and re-disabling it fast. Unfortunately it gets removed after a repaint event, so I cannot take a screenshot of it.