Review Request: Add navigation through history of sent messages in message editor
Alex Tsi
alexhs.tsi at gmail.com
Tue Aug 28 01:30:08 UTC 2012
Actually i cannot find usability in this too. In konversation, you give
commands so it might be useful.
However when chatting the only time you will probably need this is when you
make a mistake and want to resend message correct. But then again, usually
used is the correct_message_here* schema.
>On Tuesday, August 28, 2012 01:33:45 AM Martin Klapetek wrote:
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106245/#review18147
> -----------------------------------------------------------
>
>
> Just out of curiosity - what's the use case of having the last sent
> messages available on key_arrow_up press (in 1-1 chat)?
>
> - Martin Klapetek
>
> On Aug. 27, 2012, 10:19 p.m., Dan Vratil wrote:
> > -----------------------------------------------------------
> > This is an automatically generated e-mail. To reply, visit:
> > http://git.reviewboard.kde.org/r/106245/
> > -----------------------------------------------------------
> >
> > (Updated Aug. 27, 2012, 10:19 p.m.)
> >
> >
> > Review request for Telepathy.
> >
> >
> > Description
> > -------
> >
> > This implements navigation through history of sent messages by pressing
> > Up and Down arrow keys like in Konversation. It's actually based on code
> > from Konversation, I just didn't implement navigating by scrolling mouse
> > wheel in the message editor as it's possible to have scrollbar in the
> > message box (unlike in Konversation) and it would be extremely annoying
> > for users if scrolling "beyond" end of the text would change the content
> > of the message box.
> >
> > I also tried populating the history of newly opened chat by a few last
> > lines from last log, but I'm unsure about this feature. Subjectively it
> > felt a bit confusing when I opened a new window and pressing Up key was
> > pasting some text that appeared from "nowhere" and was not related to
> > this session/conversation. But feel free to discuss, I can provide the
> > code if you want to try. Alternatively we can just wait if users will
> > ask for this functionality :)
> >
> >
> > This addresses bug 303648.
> >
> > http://bugs.kde.org/show_bug.cgi?id=303648
> >
> > Diffs
> > -----
> >
> > lib/chat-text-edit.h 3e2dec0
> > lib/chat-text-edit.cpp e4ca62c
> >
> > Diff: http://git.reviewboard.kde.org/r/106245/diff/
> >
> >
> > Testing
> > -------
> >
> >
> > Thanks,
> >
> > Dan Vratil
More information about the KDE-Telepathy
mailing list