[kde-edu]: Review Request: Add variable management panel in Cantor

[Alexander Rieder]

> On 2010-09-16 07:51:08, Miha Cancula wrote:
> > That's a very nice thing, I'm sorry I didn't comment earlier, but I really didn't have much to say. It seems it took some work, but the result is really useful. 
> > 
> > I like it, especially the fact that there's no SessionExtension needed, only a normal one and a standard Model.  
> > 
> > I'm wondering if there should be a DefaultModel in the library that would implement most common virtual functions, and have an API for convenience. I'm willing to work on that, if you wish.
> > 
> > Also, I'll be implementing it in Octave. Anyway, great work.

Yes, a default model with some convenience API would be nice


- Alexander


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


On 2010-09-15 22:54:50, Alexander Rieder wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5354/
> -----------------------------------------------------------
> 
> (Updated 2010-09-15 22:54:50)
> 
> 
> Review request for KDE-Edu.
> 
> 
> Summary
> -------
> 
> Implement basics for a VariableManager panel in Cantor.
> This change is rather big, as it involves some completely new plugin interfaces to make it as easy as possible to add similar things later.
> This patch only contains a simple GUI and a dummy implementation in the NullBackend. (to test it start cantor with --backend=nullbackend), and of course the new interfaces.
> First it introduces Extensions for the Session (so far they were only for the Backend) to allow optional hooks into the workings inside the session.
> Also it introduces so called PanelPlugins, these are plugins that create widgets that are shown in the side panel depending on the exported extensions by the session.
> The current HelpPanel is turned into such a panel, and the new VariableManager is one too.
> On the backend side, one only has to implement the VariableManagerExtension, and emit the signals appropriately/implement the functions there.
> I would really like some feedback on this API (for both VariableManagerExtension and PanelPlugin side), please have a look, and possibly suggest better namings than PanelPlugin or SessionExtension
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdeedu/cantor/src/CMakeLists.txt 1174987 
>   /trunk/KDE/kdeedu/cantor/src/backends/kalgebra/CMakeLists.txt 1174987 
>   /trunk/KDE/kdeedu/cantor/src/backends/kalgebra/kalgebrabackend.cpp 1174987 
>   /trunk/KDE/kdeedu/cantor/src/backends/kalgebra/kalgebraextensions.h PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/backends/kalgebra/kalgebraextensions.cpp PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/backends/kalgebra/kalgebrasession.h 1174987 
>   /trunk/KDE/kdeedu/cantor/src/backends/kalgebra/kalgebrasession.cpp 1174987 
>   /trunk/KDE/kdeedu/cantor/src/backends/null/nullsession.h 1174987 
>   /trunk/KDE/kdeedu/cantor/src/backends/null/nullsession.cpp 1174987 
>   /trunk/KDE/kdeedu/cantor/src/backends/null/nullvariablemgrextension.h PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/backends/null/nullvariablemgrextension.cpp PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/cantor.h 1174987 
>   /trunk/KDE/kdeedu/cantor/src/cantor.cpp 1174987 
>   /trunk/KDE/kdeedu/cantor/src/cantor_part.h 1174987 
>   /trunk/KDE/kdeedu/cantor/src/cantor_part.cpp 1174987 
>   /trunk/KDE/kdeedu/cantor/src/lib/CMakeLists.txt 1174987 
>   /trunk/KDE/kdeedu/cantor/src/lib/backend.h 1174987 
>   /trunk/KDE/kdeedu/cantor/src/lib/backend.cpp 1174987 
>   /trunk/KDE/kdeedu/cantor/src/lib/cantor_panelplugin.desktop PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/lib/extension.h 1174987 
>   /trunk/KDE/kdeedu/cantor/src/lib/extension.cpp 1174987 
>   /trunk/KDE/kdeedu/cantor/src/lib/panelplugin.h PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/lib/panelplugin.cpp PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/lib/panelpluginhandler.h PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/lib/panelpluginhandler.cpp PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/lib/session.h 1174987 
>   /trunk/KDE/kdeedu/cantor/src/lib/session.cpp 1174987 
>   /trunk/KDE/kdeedu/cantor/src/panelplugins/CMakeLists.txt PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/panelplugins/helppanel/CMakeLists.txt PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/panelplugins/helppanel/helppanelplugin.cpp PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/panelplugins/helppanel/helppanelplugin.desktop PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/panelplugins/helppanel/helppanelplugin.h PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/panelplugins/variablemgr/CMakeLists.txt PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/panelplugins/variablemgr/newvardlg.ui PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/panelplugins/variablemgr/variablemanagerplugin.h PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/panelplugins/variablemgr/variablemanagerplugin.cpp PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/panelplugins/variablemgr/variablemanagerplugin.desktop PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/panelplugins/variablemgr/variablemanagerwidget.h PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/panelplugins/variablemgr/variablemanagerwidget.cpp PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/src/worksheet.cpp 1174987 
> 
> Diff: http://svn.reviewboard.kde.org/r/5354/diff
> 
> 
> Testing
> -------
> 
> This is mostly a preview for the new APIs, but all the functionality should work as it used to (I didn't notice any regressions).
> The variable management works for the dummy implementation in the NullBackend
> 
> 
> Screenshots
> -----------
> 
> The Variable Manager plugin
>   http://svn.reviewboard.kde.org/r/5354/s/503/
> 
> 
> Thanks,
> 
> Alexander
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-edu/attachments/20100916/6b47796c/attachment.htm