D17626: Print preview for KmPlot

Albert Astals Cid noreply at phabricator.kde.org
Tue Jan 15 23:20:44 GMT 2019


aacid added a comment.


  Then it looks ok to me (well there's no need to move some of the includes to the header, with fordward declarations it would be enough, but who cares i guess :D)
  
  Let's see if @tcanabrava has something else to say :)

INLINE COMMENTS

> cfeck wrote in maindlg.cpp:235
> There is no `addAction(KStandardAction::StandardAction, QString, QObject, Func)` prototype in `KActionCollection`, so it's OK.

oh, someone should fix that :)

REPOSITORY
  R334 KmPlot

REVISION DETAIL
  https://phabricator.kde.org/D17626

To: yurchor, #kde_edu, tcanabrava
Cc: cfeck, aacid, tcanabrava, kde-doc-english, kde-edu, skadinna, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20190115/f86828ba/attachment.html>


More information about the kde-edu mailing list