Review Request: Make it possible to start conversations from the Quick Chat plasmoid

[Lasath Fernando]

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


Okay, I kind of like where this is going. In fact, it makes me think that we should consider merging the chat plasmoid and contact list plasmoid. But I'll expand on that in a separate ML thread.

As for where this plasmoid is/should be going, I'll elaborate my original idea, and then we can implement it or discuss alternatives. I originally had in mind 2 ways of using this:

1. A Global Switch - Somewhere in a settings page or install configuration there would be a flag that decides whether to use this plasmoid for _all_ chats. Once we get all the showstopper bugs fixed, I'd like to use this on my laptop, since it only has 1920*1080 pixels of total screen space, and this plasmoid uses up significantly less than the standard text-ui.

2. Specific Contacts/Groups - The idea with this was to allow the user to choose a list of contacts or groups (if we ever get a 'most contacted' group, that would be a brilliant candidate) in settings somewhere. Then this would go before the approver and if the incoming Tp::TextChannel was for a contact in said list, it would open with this; otherwise it would pass it to the text-ui.

- Lasath Fernando


On Dec. 19, 2012, 3:10 a.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107811/
> -----------------------------------------------------------
> 
> (Updated Dec. 19, 2012, 3:10 a.m.)
> 
> 
> Review request for Telepathy and David Edmundson.
> 
> 
> Description
> -------
> 
> I've wanted to use this plasmoid for longtime, I fear that the reason why I'm not using it yet is because I can't start chats from there. The plan is that this will improve the situation, to some extent.
> 
> The important part about the patch is that the root element is refactored into a grid that can have different elements. Now I added a button with a contactlist that probably should go, but eventually i'd like to have non-conversation buttons so that one can pin contacts.
> 
> 
> Diffs
> -----
> 
>   plasmoid/declarative-plugin/conversations-model.h f9dc047 
>   plasmoid/declarative-plugin/conversations-model.cpp faaa60b 
>   plasmoid/org.kde.ktp-chatplasmoid/contents/ui/ContactList.qml PRE-CREATION 
>   plasmoid/org.kde.ktp-chatplasmoid/contents/ui/ConversationDelegate.qml 8a8d851 
>   plasmoid/org.kde.ktp-chatplasmoid/contents/ui/ConversationDelegateButton.qml PRE-CREATION 
>   plasmoid/org.kde.ktp-chatplasmoid/contents/ui/main.qml feb766b 
> 
> Diff: http://git.reviewboard.kde.org/r/107811/diff/
> 
> 
> Testing
> -------
> 
> Very little, mostly sending this review for starting a discussion on where we'd like to go with this plasmoid.
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

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