Review Request 117177: Add action to open contact list from ktp-text-ui

[Siddhartha Sahu]

> On March 30, 2014, 4:30 p.m., David Edmundson wrote:
> > Where does this add it in the UI? Is it an optional button in the toolbar?
> > Maybe it should be in the menu too.
> > 
> > Thomas, any opinions?
> 
> Thomas Pfeiffer wrote:
>     First of all: What user story does this feature have? In which situation would users want to open the contact list from the chat window? Once we have that defined, we can think about how to implement it in the UI
> 
> Siddhartha Sahu wrote:
>     @David: Yes it is an optional button which the user will have to manually add it to the toolbar. Adding it to the menu sounds good too.
>     
>     @Thomas: This patch is for https://bugs.kde.org/show_bug.cgi?id=307659.
>     Cases in which I think it will be helpful are when user is not using the desktop applet for some reason, or to quickly open the contact list from the chat window (eg using a shortcut).
>     
>     Personally now that I'm using it, I find it very convenient being able to open the contact list (eg. after I receive a new message) without having to move the mouse all over to the system tray
> 
> Martin Klapetek wrote:
>     I'm just curious - why do you open contact list after you receive new message?
> 
> Siddhartha Sahu wrote:
>     Receiving a message from a friend asking about something, which I then ask to another friend ^_^
> 
> David Edmundson wrote:
>     For that would the following then be better:
>      - A button "start new chat" in the menu
>      - This opens a dialog with a list of all contacts
>      - Selecting here starts a chat
>     
>     ?
> 
> Martin Klapetek wrote:
>     In other words, use KRunner? :)

Of course, we already have many ways to open the contact list, this is just another option. I added it only after seeing the bug report, so I'm not arguing too strongly, but I found it a good to have ^_^


- Siddhartha


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117177/#review54583
-----------------------------------------------------------


On March 30, 2014, 10:31 a.m., Siddhartha Sahu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117177/
> -----------------------------------------------------------
> 
> (Updated March 30, 2014, 10:31 a.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Bugs: 313404
>     http://bugs.kde.org/show_bug.cgi?id=313404
> 
> 
> Repository: ktp-text-ui
> 
> 
> Description
> -------
> 
> Will add documentation on userbase once committed :)
> 
> 
> Diffs
> -----
> 
>   app/chat-window.cpp 2c2e12b96620ad180f0269aff66fb860c0bb6fb5 
>   app/chat-window.h 793ac559cf3e640ca653dd60ba04c0fdbb264da5 
> 
> Diff: https://git.reviewboard.kde.org/r/117177/diff/
> 
> 
> Testing
> -------
> 
> Contact list opens
> 
> 
> Thanks,
> 
> Siddhartha Sahu
> 
>

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