[kde-edu]: Review Request: [Cantor] Improved Script editor window

Miha Cancula miha.cancula at gmail.com
Mon May 3 23:12:27 CEST 2010

This is an automatically generated e-mail. To reply, visit:

Review request for KDE-Edu and Alexander Rieder.


Change the script editor to use XmlGui for actions, with complete KTextEditor toolbars and menus.

The patch is a little rough still, with unneeded whitespace changes, I will probably update it soon. There are also some leftover debug calls and error checks. You can see the code and comment on it. I will also explain and/or correct any wtf-moments that may have occured. 

I realize that trunk is in a soft treeze right now, and that this isn't a killer feature, so I think there's no rush to get it commited. I would like some feedback on it.


  /trunk/KDE/kdeedu/cantor/src/CMakeLists.txt 1120620 
  /trunk/KDE/kdeedu/cantor/src/cantor.cpp 1120620 
  /trunk/KDE/kdeedu/cantor/src/cantor_part.h 1120620 
  /trunk/KDE/kdeedu/cantor/src/cantor_part.cpp 1120620 
  /trunk/KDE/kdeedu/cantor/src/cantor_scripteditor.rc PRE-CREATION 
  /trunk/KDE/kdeedu/cantor/src/scripteditorwidget.h 1120620 
  /trunk/KDE/kdeedu/cantor/src/scripteditorwidget.cpp 1120620 

Diff: http://reviewboard.kde.org/r/3878/diff


The GUI looks and behaves properly. I tried some different ways of closing the script editor and the running session and it behaved as expected in all of them. 

It was tested with last week's trunk.


Script editor with KTextEditor's menus



More information about the kde-edu mailing list