Review Request: Baby steps for Chat Plasmoid...

Lasath Fernando kde at lasath.org
Wed Jan 4 13:47:28 UTC 2012


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

(Updated Jan. 4, 2012, 1:47 p.m.)


Review request for Telepathy and David Edmundson.


Description
-------

Right, I decided that I should have reviewed and/or merge what work I've done so far rather than letting it pile up in its corner into a giant worthless clump. 

To the people that haven't heard (or wasn't at woshibon), this is a chat plasmoid that more or less behaves like facebook and google talk, except it sits in your taskbar :)

And in terms of feedback, at this stage I think design issues should take priority over sane code because the main reason I'm doing this is because I don't want to have to do any massive restructuring later on.

And if things don't make sense, ask me (I didn't comment/document anything well and I'm certainly too sleepy now ;)

PS: All the code is in my scratch repo
http://quickgit.kde.org/?p=clones%2Ftelepathy-text-ui%2Ffernando%2Fqmlplugin.git&a=shortlog&h=refs/heads/qml_plugins2


Diffs
-----

  CMakeLists.txt d1cc185 
  lib/CMakeLists.txt 5d39a62 
  lib/chat-window-style.cpp 7b770cf 
  lib/conversation-que-manager.h PRE-CREATION 
  lib/conversation-que-manager.cpp PRE-CREATION 
  lib/conversation-target.h PRE-CREATION 
  lib/conversation-target.cpp PRE-CREATION 
  lib/conversation.h PRE-CREATION 
  lib/conversation.cpp PRE-CREATION 
  lib/conversations-model.h PRE-CREATION 
  lib/conversations-model.cpp PRE-CREATION 
  lib/messages-model.h PRE-CREATION 
  lib/messages-model.cpp PRE-CREATION 
  lib/qml-plugins.h PRE-CREATION 
  lib/qml-plugins.cpp PRE-CREATION 
  lib/qmldir PRE-CREATION 
  lib/telepathy-text-observer.h PRE-CREATION 
  lib/telepathy-text-observer.cpp PRE-CREATION 
  plasmoid/CMakeLists.txt PRE-CREATION 
  plasmoid/contents/ui/ChatWidget.qml PRE-CREATION 
  plasmoid/contents/ui/ConversationDelegate.qml PRE-CREATION 
  plasmoid/contents/ui/TextDelegate.qml PRE-CREATION 
  plasmoid/contents/ui/main.qml PRE-CREATION 
  plasmoid/metadata.desktop PRE-CREATION 

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


Testing
-------

Um.. yeah... about that...    :/


Thanks,

Lasath Fernando

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20120104/a847b50b/attachment.html>


More information about the KDE-Telepathy mailing list