Review Request: Add main menu to logviewer

Dan Vrátil dvratil at redhat.com
Wed Nov 21 15:55:45 UTC 2012


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

Review request for Telepathy.


Description
-------

This patch adds following main menu to the logviewer main window:

File
   - Quit
Logs
   - Clear account history
   - Clear contact history
   -----------------------
   - Import Kopete Logs
Conversation
   - Previous Conversation (Ctrl+P)
   - Next Conversation (Ctrl+N)
Settings
Help

(The last two menus contain default KXmlGuiWindow actions, I decided to omit the "Glorify dvratil" action).

I converted the LogViewer class from QWidget to KXmlGuiWindow and since I was already digging in the code, I moved the Tp Factories initialization to main(), renamed some slots, moved some code etc - no major changes, it just makes me happier when reading the code.

The new feature (because of which I decided to do this all) are "Previous Conversation" and "Next Conversation" actions. I'm bored scrolling all the way up in long chats to reach the "Previous Conversation" link on the beginning of the log. Having these two actions allows me to do this just by using keyboard shortcuts ;)


Diffs
-----

  logviewer/CMakeLists.txt 3953dc1 
  logviewer/log-viewer.h 8829e7b 
  logviewer/log-viewer.cpp 13db0c4 
  logviewer/log-viewer.rc PRE-CREATION 
  logviewer/main.cpp e7eb0b5 

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


Testing
-------

Looks like everything still works.


Thanks,

Dan Vrátil

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


More information about the KDE-Telepathy mailing list