Review Request: [logviewer] Add links to previous and next conversations to empty logs too

Dan Vratil dan at progdan.cz
Mon Aug 6 19:38:46 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105762/
-----------------------------------------------------------

(Updated Aug. 6, 2012, 7:38 p.m.)


Review request for Telepathy.


Changes
-------

Simplify methods to find next and previous date.

(thanks David, I didn't know about the q*Bound functions)


Description
-------

Add "Previous conversation" and "Next conversation" links to logview even when the chat log for selected date is empty, so that it's easy to jump to nearest date with some non-empty chat log.


This addresses bug 304137.
    http://bugs.kde.org/show_bug.cgi?id=304137


Diffs (updated)
-----

  logviewer/conversation-date-picker.cpp c6bbf2e 
  logviewer/log-viewer.cpp 5866cdb 

Diff: http://git.reviewboard.kde.org/r/105762/diff/


Testing
-------


Thanks,

Dan Vratil

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20120806/cce4cb66/attachment-0001.html>


More information about the KDE-Telepathy mailing list