[kde-edu]: Review Request: [KAnagram] Ported to Get Hot new Stuff 3

[etienne.rebetez at oberwallis.ch]

> On 2010-04-04 10:46:36, Frederik Gladhorn wrote:
> > /trunk/KDE/kdeedu/kanagram/src/vocabsettings.cpp, line 101
> > <http://reviewboard.kde.org/r/3485/diff/2/?file=22508#file22508line101>
> >
> >     refresh view seems to take forever. maybe just call it if the user actually changed something? if (dialog.changedEntries().size() > 0)

Done;)


- eti


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


On 2010-04-04 15:52:44, eti wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/3485/
> -----------------------------------------------------------
> 
> (Updated 2010-04-04 15:52:44)
> 
> 
> Review request for KDE-Edu and Frederik Gladhorn.
> 
> 
> Summary
> -------
> 
> Finally i found some time to finish my homework from the last klassroom;)
> KAnagram is now ported to "Get hot new Stuff 3". 
> I also did a small UI change in the configuration dialog.
> 
> Let me know if you like it.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdeedu/kanagram/src/CMakeLists.txt 1110654 
>   /trunk/KDE/kdeedu/kanagram/src/kanagram.cpp 1110654 
>   /trunk/KDE/kdeedu/kanagram/src/kanagram.knsrc 1110654 
>   /trunk/KDE/kdeedu/kanagram/src/newstuff.h 1110509 
>   /trunk/KDE/kdeedu/kanagram/src/newstuff.cpp 1110509 
>   /trunk/KDE/kdeedu/kanagram/src/newstuffwidget.ui 1110509 
>   /trunk/KDE/kdeedu/kanagram/src/vocabsettings.h 1110654 
>   /trunk/KDE/kdeedu/kanagram/src/vocabsettings.cpp 1110654 
>   /trunk/KDE/kdeedu/kanagram/src/vocabsettingswidget.ui 1110654 
> 
> Diff: http://reviewboard.kde.org/r/3485/diff
> 
> 
> Testing
> -------
> 
> Somehow deinstalling the vocas dosn't work. They remain in the home folder.
> Is this a GHNS3 Bug or did i forgot something?
> 
> 
> Thanks,
> 
> eti
> 
>