Review Request 120838: Fix quit action by connecting it to qApp instead of kapp.

Jeremy Whiting jpwhiting at kde.org
Tue Oct 28 05:49:49 UTC 2014


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

Review request for KDE Edu and Laurent Navet.


Repository: klettres


Description
-------

KLettres doesn't use KApplication anymore, so connect to
the QApplication's quit slot instead.


Diffs
-----

  src/klettres.cpp 0390a52b2ef425277d9b19a4ef1e2e9876a585df 

Diff: https://git.reviewboard.kde.org/r/120838/diff/


Testing
-------

File -> Quit now actually does what it should.

Seems a lot of the stuff in setupActions isn't showing in the ui yet, something got missed when porting to frameworks I guess.


Thanks,

Jeremy Whiting

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20141028/74590124/attachment.html>


More information about the kde-edu mailing list