> On March 30, 2014, 11 a.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? :) > > Siddhartha Sahu wrote: > 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 ^_^
Oh, so the idea was so that a shortcut works. (Sorry I really should read bug links) - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117177/#review54583 ----------------------------------------------------------- On March 30, 2014, 5:01 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, 5:01 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 > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
