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