Bug 160561 - [UI] Hide Offline Users option (Settings) is counterintuitive
Summary: [UI] Hide Offline Users option (Settings) is counterintuitive
Status: RESOLVED FIXED
Alias: None
Product: kopete
Classification: Unmaintained
Component: Contact list (show other bugs)
Version: unspecified
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: Kopete Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-04-08 15:16 UTC by Martin Fitzpatrick
Modified: 2008-05-11 20:34 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Fitzpatrick 2008-04-08 15:16:30 UTC
Version:            (using KDE 4.0.3)
Installed from:    Ubuntu Packages
OS:                Linux

Under the Settings menu is an option to Show/Hide Offline Users. Clicking on this option in the menu toggles the menu entry to show "Show Offline Users" (while they are hidden) or "Hide Offline Users" (while they are shown). However, there is also a tickbox which confuses the meaning of the menu entry. So, there are two states:

[ ] Show Offline Users
[x] Hide Offline Users

In the first state (offline users currently hidden) it is obvious that to show them you click, and in doing so you would expect the menu entry to change to [x] Show Offline Users to indicate this. However, it changes to the 2nd state:

In the 2nd state it looks as if you have "select" to Hide Offline Users (checkbox ticked). In this state you would expect that clicking the option would UNhide Offline Users by removing the check. Instead clicking this option Hides Offline Users.

Expected behaviour:
(a) The menu item should not have a checkbox and should toggle between the two states.
or
(b) The menu item should have a checkbox, but the menu text should not change.

Thanks :)
Comment 1 FiNeX 2008-04-12 00:06:35 UTC
I agree with Margin. Change the label of the option is not a good idea.
Comment 2 Dennis Nienhüser 2008-05-11 20:05:43 UTC
SVN commit 806558 by nienhueser:

No need to emulate toggle actions anymore, use the real thing and leave the text alone :)
Backport of commit 784900 by ahartmetz
BUG: 160561


 M  +2 -0      kopetewindow.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=806558
Comment 3 FiNeX 2008-05-11 20:34:18 UTC
Thanks Dennis :-)