Review Request: Refactor filterOfflineUsers to showOfflineUsers
Rohan Garg
rohangarg at kubuntu.org
Wed Jun 15 15:27:58 CEST 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101630/
-----------------------------------------------------------
(Updated June 15, 2011, 1:27 p.m.)
Review request for Telepathy.
Changes
-------
Fix summary
Summary (updated)
-------
Refactor filterOfflineUsers to showOfflineUsers
- This commit changes the nature of the Show/Hide Offline Users action button
- Pushing the button in now shows the offline users instead of hiding them
- String change : "Show/Hide offline users" -> "Hide/Show offline users"
Diffs
-----
account-filter-model.h 007c2a9881141ce1999dd1d598a619d2f7e5fd8b
account-filter-model.cpp d30b20861b4dd9853b543df17b7f8475cf7865df
main-widget.h abee064e9c2d3353f6f2d7bfebc32a61a004f0df
main-widget.cpp 65fff9326aed047e6fd815586f12e310c02b2595
Diff: http://git.reviewboard.kde.org/r/101630/diff
Testing
-------
Yes, clicking the "Hide/Show offline users" now shows the offline users when it's in the 'Pressed' state, does not show them otherwise
Thanks,
Rohan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20110615/71342a82/attachment.htm
More information about the KDE-Telepathy
mailing list