Review Request: Add option to set how many lines of history should be loaded in chat
Dan Vrátil
dvratil at redhat.com
Fri Dec 28 02:10:24 UTC 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107968/
-----------------------------------------------------------
(Updated Dec. 28, 2012, 2:10 a.m.)
Review request for Telepathy.
Description
-------
Adds a QSpinBox to "Chat Tab Behavior" tab to set how many lines of previous conversation should be loaded when opening a new chat.
The default amount is 4, available range is 0 - 100. I think that this is better then "Last hour", "Last 24 hours" etc as Daniele suggested in the bug report because "Last 24 hours" can result in no message or 1000 messages being loaded.
It also fixes the LogManager to return m_fetchAmount instead of m_fetchAmount + 1 messages :)
This addresses bug 310982.
http://bugs.kde.org/show_bug.cgi?id=310982
Diffs (updated)
-----
config/behavior-config.h 55d2fea
config/behavior-config.cpp 6e78168
config/behavior-config.ui 7b8e304
lib/chat-widget.cpp 8f63e14
lib/logmanager.cpp 50d1622
Diff: http://git.reviewboard.kde.org/r/107968/diff/
Testing
-------
Thanks,
Dan Vrátil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20121228/6736966e/attachment.html>
More information about the KDE-Telepathy
mailing list