Review Request: Add account online-ness to the model.

David Edmundson kde at davidedmundson.co.uk
Wed Jan 12 02:14:25 CET 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100369/
-----------------------------------------------------------

Review request for Telepathy.


Summary
-------

Added 3 new roles to the account model. Connection as a string (online, connecting, disconnected), and as an icon.
Also added a string which says why the user is offline.
(because the account is disabled, because the password is wrong, network error etc)

Then created an Item Delegate for the accounts view that looks pretty.


Diffs
-----

  src/CMakeLists.txt 6ebed1268e697b9816ac0620467605c71324b3ef 
  src/KCMTelepathyAccounts/account-edit-widget.h 4c78338603dc0b88230b52190ca9d373dff2628d 
  src/KCMTelepathyAccounts/account-edit-widget.cpp b554db71a17a4c9664d04f781561ce62cbc17562 
  src/KCMTelepathyAccounts/account-edit-widget.ui 6387e2be80747b8721bef9934258d2d489cd8cac 
  src/KCMTelepathyAccounts/parameter-edit-widget.cpp 187c39fbf990b610b8985d1869d48bc2be7c9c6d 
  src/KCMTelepathyAccounts/parameter-edit-widget.ui 25a2453d5a85ebe3f646c28e7acdb728ca14f1fa 
  src/KCMTelepathyAccounts/protocol-item.h 90b96ee4ac6f4185a74a4d197b47ee1830564a43 
  src/KCMTelepathyAccounts/protocol-item.cpp 03ac0bceff0f32673a706329f698a66323842c69 
  src/KCMTelepathyAccounts/protocol-select-widget.cpp fa4b3c6d49d2287b63ba64730e1ec034508e8c55 
  src/account-item.h 75e24116c525e2315c9d8f622d3b2fcb1c5c1bb8 
  src/account-item.cpp 50afaa54c96ad51b6775f954cc180d1e12bbe96b 
  src/accounts-list-delegate.h PRE-CREATION 
  src/accounts-list-delegate.cpp PRE-CREATION 
  src/accounts-list-model.h 815719a08b1a3091d007a6ef30ab1131fd8b489c 
  src/accounts-list-model.cpp e0cf9bd13c7f1af8e8d936fe3e66af6db3aefdbc 
  src/add-account-assistant.cpp 3933763957b1448aacf9e6c7efbc6b296a5e3664 
  src/edit-account-dialog.cpp 73462f2e2a93e22e6d5d67297c575f4e10d44c5c 
  src/kcm-telepathy-accounts.h 513db22457c2b77b4e36d54689d3b16810ac6854 
  src/kcm-telepathy-accounts.cpp c2579d2eb38c371f23e86f0a468f1ce90ef75b88 
  src/main-widget.ui b8ad1e5e243f923240382eda22414347ae173907 

Diff: http://git.reviewboard.kde.org/r/100369/diff


Testing
-------

Opened concurrently in both empathy and the accounts KCM, checked online state changed correctly, and that the "enabled" flag worked.

Tried accounts with network errors and incorrect passwords.


Screenshots
-----------


  http://git.reviewboard.kde.org/r/100369/s/42/


Thanks,

David

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20110112/1117a514/attachment.htm 


More information about the KDE-Telepathy mailing list