Review Request: Stage: GSoC: New GUI for manage custom slide shows

[Paul Mendez]

> On July 19, 2011, 1:02 p.m., Thorsten Zachmann wrote:
> > Hello Paul, 
> > I did not yet look at the code. However I have tested the patch and found the following:
> > 
> > When changing the zoom level in the slide sorted only the zoom in the upper window changes not the one on the custom slide show.  Both lists should be updated when changing the zoom. When the zoom is changed and then you move a item from up to down the position indicator is wrong. This is related to the not zooming of the lower (custom slide show) 
> >
> 
> Paul Mendez wrote:
>     I think that I have already fix that in the last patch, but if you find that behavior again it could be a different bug, and you could tell me how reproduce it, because with the last patch the zoom is working as expected.
> 
> Thorsten Zachmann wrote:
>     I get that behaviour with the latest patch that is attached to the reviewboard. To reproduce the problem do the folliwing:
>     
>     1. Open stage with a presentation containing some slides.
>     2. OPen the slides sorter
>     3. Create a new custom slide show
>     4. Move some slides to the custom slide show
>     5. Zoom to 150% with the slider at the bottom.
>     -> only the in the upper part which contains all slides the slides are resized, the slides in the custom slideshow keep their size
>     see http://www.zagge.de/files/stage_customslideshow.png for the screenshot showing the problem

Sorry, You were right the last patch was incomplete. I left some changes in other branch. I have updated the patch, now I expect that the zoom issue is resolved.


- Paul


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


On July 20, 2011, 10:32 p.m., Paul Mendez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101820/
> -----------------------------------------------------------
> 
> (Updated July 20, 2011, 10:32 p.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Summary
> -------
> 
> (preliminary) new GUI for manage custom slide shows. 
> Note: As some GUI integration is still missing, it's only a review request, it isn't for merge with master branch.
> (part of GSoC project)
> 
> 
> Diffs
> -----
> 
>   kpresenter/part/CMakeLists.txt 613ff8d3de84a19776df1651c0bd3604fc395099 
>   kpresenter/part/KPrAnimationDirector.cpp d9600e220546f5d5ed29f73e1988f3d130f69770 
>   kpresenter/part/KPrCustomSlideShows.h 7d8a91b67e14e32312fd2706dcede6efac6e1aa7 
>   kpresenter/part/KPrCustomSlideShows.cpp 632dda6b772417b3d26a84992352f9caa33ca158 
>   kpresenter/part/KPrCustomSlideShowsModel.h PRE-CREATION 
>   kpresenter/part/KPrCustomSlideShowsModel.cpp PRE-CREATION 
>   kpresenter/part/KPrDocument.h 7756f5e542738232bb3ea278453319a71d28ce9d 
>   kpresenter/part/KPrDocument.cpp 02bf53cf087746f1942adad5db3f1e4d30a1b618 
>   kpresenter/part/KPrSlidesManagerView.h 6f345e049cdf746e00fdfd9e3e7644668ff9e96e 
>   kpresenter/part/KPrSlidesManagerView.cpp 39e80b1b770584f97e6bfcbcebc06bd107bc1a68 
>   kpresenter/part/KPrSlidesSorterDocumentModel.h a5b65a4d87f8af6a51b4e1992c2ac2a3d23aed37 
>   kpresenter/part/KPrSlidesSorterDocumentModel.cpp 4938e3696bed3fb90c017aba5997c40268ce3e99 
>   kpresenter/part/KPrSlidesSorterItemDelegate.h PRE-CREATION 
>   kpresenter/part/KPrSlidesSorterItemDelegate.cpp PRE-CREATION 
>   kpresenter/part/KPrView.h a558a2e171731a2390bf5a709a829132082b993b 
>   kpresenter/part/KPrView.cpp 04d502d4d61551ad0bd6dfabff9b8237e378b38c 
>   kpresenter/part/KPrViewModePresentation.cpp 46b61e1de8a9bdcab8bcb81ac3ea2830ff6f2e24 
>   kpresenter/part/KPrViewModeSlidesSorter.h 240bd9e0c6b4158ff0726f2e72e4c79982a9ea18 
>   kpresenter/part/KPrViewModeSlidesSorter.cpp 61866f8a1344e1999224444d71a46a1a21a1864e 
>   kpresenter/part/commands/KPrAddCustomSlideShowCommand.h PRE-CREATION 
>   kpresenter/part/commands/KPrAddCustomSlideShowCommand.cpp PRE-CREATION 
>   kpresenter/part/commands/KPrDelCustomSlideShowCommand.h PRE-CREATION 
>   kpresenter/part/commands/KPrDelCustomSlideShowCommand.cpp PRE-CREATION 
>   kpresenter/part/commands/KPrDeleteSlidesCommand.h PRE-CREATION 
>   kpresenter/part/commands/KPrDeleteSlidesCommand.cpp PRE-CREATION 
>   kpresenter/part/commands/KPrEditCustomSlideShowsCommand.h PRE-CREATION 
>   kpresenter/part/commands/KPrEditCustomSlideShowsCommand.cpp PRE-CREATION 
>   kpresenter/part/commands/KPrRenameCustomSlideShowCommand.h PRE-CREATION 
>   kpresenter/part/commands/KPrRenameCustomSlideShowCommand.cpp PRE-CREATION 
>   kpresenter/part/commands/KPrSetCustomSlideShowsCommand.h 6cc29e2cd44ef4fc2b63efbc1c3187cfd4ccd198 
>   kpresenter/part/commands/KPrSetCustomSlideShowsCommand.cpp 24b99969ce4cbe55a480e43e98d2ae370c20e721 
>   kpresenter/part/tests/CMakeLists.txt f9b56a6ed55ebe610ae30cd21437975c00c317d1 
>   kpresenter/part/tests/PAMock.h PRE-CREATION 
>   kpresenter/part/tests/TestAddCustomSlideShowCommand.h PRE-CREATION 
>   kpresenter/part/tests/TestAddCustomSlideShowCommand.cpp PRE-CREATION 
>   kpresenter/part/tests/TestDelCustomSlideShowCommand.h PRE-CREATION 
>   kpresenter/part/tests/TestDelCustomSlideShowCommand.cpp PRE-CREATION 
>   kpresenter/part/tests/TestEditCustomSlideShowsCommand.h PRE-CREATION 
>   kpresenter/part/tests/TestEditCustomSlideShowsCommand.cpp PRE-CREATION 
>   kpresenter/part/tests/TestRenameCustomSlideShowCommand.h PRE-CREATION 
>   kpresenter/part/tests/TestRenameCustomSlideShowCommand.cpp PRE-CREATION 
>   kpresenter/part/ui/KPrConfigureSlideShowDialog.h 1c7794a2ff08e096b8a86cf2673db0e6f206eeb6 
>   kpresenter/part/ui/KPrConfigureSlideShowDialog.cpp 847a5a54a08fb46dab34bbd828da8d1ab21c6342 
>   kpresenter/part/ui/KPrCustomSlideShowsDialog.h f5d53f90ca2d1e7c53ac4779a751a8d58e384a0c 
>   kpresenter/part/ui/KPrCustomSlideShowsDialog.cpp da697e04426c8c8c4d8684af1da9fd10c7faac61 
>   kpresenter/part/ui/KPrCustomSlideShowsDialog.ui be25bc8932809e41a139f364478d2097eaa78110 
>   libs/kopageapp/KoPADocument.h 9347ae9c8a302c0dc6c0139c2d1e0951cd5a6091 
>   libs/kopageapp/KoPADocument.cpp 2a03e42fde69f5d4b0b144ccbde915ef826de70d 
>   libs/kopageapp/KoPADocumentStructureDocker.cpp 02af38237e08e075934cd5110efe59e56234a9e2 
> 
> Diff: http://git.reviewboard.kde.org/r/101820/diff
> 
> 
> Testing
> -------
> 
> build and run
> 
> 
> Thanks,
> 
> Paul
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20110720/1127d1a5/attachment.htm>