[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-edu-devel
Subject:    Re: [kde-edu]: Review Request: [Cantor] Improved Script editor
From:       "Miha Cancula" <miha.cancula () gmail ! com>
Date:       2010-05-07 19:46:33
Message-ID: 20100507194633.7721.52977 () localhost
[Download RAW message or body]



> On 2010-05-07 14:18:02, Alexander Rieder wrote:
> > Looks really good, and now as I think of it it shouldn't interfere with the \
> > soft-freeze, as its no new feature afterall, just an improvement on the old(just \
> > make sure we don't hit the hard-freeze on tuesday). Do you have an svn account to \
> > commit this? If not I suggest you to apply for one (see \
> > http://techbase.kde.org/Contribute/Get_a_SVN_Account), as I hope you'll do more \
> > work during your Season of KDE. 
> > Thank you for your work.

Thank you, I already got the account, I committed the patch as r1124106.


- Miha


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


On 2010-05-07 13:00:53, Miha Cancula wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/3878/
> -----------------------------------------------------------
> 
> (Updated 2010-05-07 13:00:53)
> 
> 
> Review request for KDE-Edu and Alexander Rieder.
> 
> 
> Summary
> -------
> 
> 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. 
> 
> Diffs
> -----
> 
> /trunk/KDE/kdeedu/cantor/src/scripteditorwidget.cpp 1123901 
> /trunk/KDE/kdeedu/cantor/src/CMakeLists.txt 1123901 
> /trunk/KDE/kdeedu/cantor/src/cantor.cpp 1123901 
> /trunk/KDE/kdeedu/cantor/src/cantor_part.h 1123901 
> /trunk/KDE/kdeedu/cantor/src/cantor_part.cpp 1123901 
> /trunk/KDE/kdeedu/cantor/src/cantor_scripteditor.rc PRE-CREATION 
> /trunk/KDE/kdeedu/cantor/src/scripteditorwidget.h 1123901 
> 
> Diff: http://reviewboard.kde.org/r/3878/diff
> 
> 
> Testing
> -------
> 
> 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.
> 
> 
> Screenshots
> -----------
> 
> Script editor with KTextEditor's menus
> http://reviewboard.kde.org/r/3878/s/385/
> 
> 
> Thanks,
> 
> Miha
> 
> 

_______________________________________________
kde-edu mailing list
kde-edu@mail.kde.org
https://mail.kde.org/mailman/listinfo/kde-edu


[prev in list] [next in list] [prev in thread] [next in thread]