Review Request 110617: Implemented Open Action for Khipu

[Punit Mehta]

> On May 24, 2013, 10:49 a.m., Aleix Pol Gonzalez wrote:
> > src/dashboard.cpp, line 90
> > <http://git.reviewboard.kde.org/r/110617/diff/1/?file=146036#file146036line90>
> >
> >     This openclicked variable should go. It's ugly and I don't see why you need it.

We need this to consider the case  : when setCurrentSpace is called , it would request for the plots only when the open is clicked. In other cases , the plots are already created . so it would not unneccessary create plots .!


- Punit


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


On May 24, 2013, 1:18 p.m., Punit Mehta wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110617/
> -----------------------------------------------------------
> 
> (Updated May 24, 2013, 1:18 p.m.)
> 
> 
> Review request for KDE Edu, Aleix Pol Gonzalez and Percy Camilo Triveño Aucahuasi.
> 
> 
> Description
> -------
> 
> The diff contains the implementation of the open action in Khipu.! The attached parent diff is for the save action which is already reviewed.! 
> If the user saves the plotspaces in .khipu file and then  he can open the file when he wants.! The initial implementation is done in a way that it considers the case only when on work is done ( i.e the user has just opened the application and then in the very next step , he is going to open his previously saved .khipu file.! I am working for dealing with the other cases also..! 
> 
> - I have added the entry of plot-colot in the .khipu file as we are requesting and creating the plots when the plotspace-thumbnail is doubleclicked.! So , we need that information also.!
> 
> 
> Diffs
> -----
> 
>   src/dashboard.h a2a9c46 
>   src/dashboard.cpp 688a49d 
>   src/mainwindow.h 1f059f8 
>   src/mainwindow.cpp 9d6e010 
> 
> Diff: http://git.reviewboard.kde.org/r/110617/diff/
> 
> 
> Testing
> -------
> 
> Done.! as mentioned , it works very well for the specified case.!
> 
> 
> Thanks,
> 
> Punit Mehta
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20130524/63f1d0b3/attachment.html>