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

@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


- 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/f0d0cea2/attachment-0001.html>