[kde-edu]: Review Request: KWordQuiz: new feature Language Tutor

[Aleks]

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


Hmm... this reviewboard is tricky, you add a comment, it gets tucked away somewhere in the middle of the page and sits there in obscurity. So I'll just re-post it to the end of the list:

I tried to change to KStandardAction, but there's a snag - I use KSystemTrayIcon::setContextMenu(QMenu) , thus my menu can be only QMenu and it's actions can be only QActions. Maybe there is a workaround, though. 

- Aleks


On 2010-02-05 08:14:35, Aleks wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/2822/
> -----------------------------------------------------------
> 
> (Updated 2010-02-05 08:14:35)
> 
> 
> Review request for KDE-Edu.
> 
> 
> Summary
> -------
> 
> A feature which pops up words randomly at certain time intervals.
> It is implemented through a -g tutor [file] command line switch.
> It creates a trayicon. When user presses start menu item, a window pops up at an interval defined in Preferences. There are two GlobalShortcut bindings, one for closing the flashcard, another for showing the reverse side.
> It might not be prudent to load the whole binary into memory if one wants to run just the Tutor, so there is an option of implemeting this Tutor as a stand-alone program.
> Also the Preferences Dialog is not yet finished and the Settings dont get saved upon exit.
> I just wanted to here people's comments and suggestions before I tidy up the code.
> 
> KWQTutor class creates a SysTray, Menu and connects menu items to SLOTs.
> startstoppressed SLOT displays the QLabel (the flashcard itself) with the text.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdeedu/kwordquiz/src/CMakeLists.txt 1084846 
>   /trunk/KDE/kdeedu/kwordquiz/src/kwordquiz.kcfg 1084846 
>   /trunk/KDE/kdeedu/kwordquiz/src/kwqtutor.h PRE-CREATION 
>   /trunk/KDE/kdeedu/kwordquiz/src/kwqtutor.cpp PRE-CREATION 
>   /trunk/KDE/kdeedu/kwordquiz/src/kwqtutordialog.h PRE-CREATION 
>   /trunk/KDE/kdeedu/kwordquiz/src/kwqtutordialog.cpp PRE-CREATION 
>   /trunk/KDE/kdeedu/kwordquiz/src/kwqtutorlabel.h PRE-CREATION 
>   /trunk/KDE/kdeedu/kwordquiz/src/main.cpp 1084846 
>   /trunk/KDE/kdeedu/kwordquiz/src/tutorform.ui PRE-CREATION 
> 
> Diff: http://reviewboard.kde.org/r/2822/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Aleks
> 
>