----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101129/#review2815 -----------------------------------------------------------
When testing I notices that centering of the page does not work after loading the file. kpresenter/part/KPrView.cpp <http://git.reviewboard.kde.org/r/101129/#comment2454> Please add a blank after the comma. kpresenter/part/KPrView.cpp <http://git.reviewboard.kde.org/r/101129/#comment2457> Center the page when zooming is wrong. The zoom should always keep the current center when zooming. (This seems to be broken at the moment) kpresenter/part/KPrViewModeSlidesSorter.cpp <http://git.reviewboard.kde.org/r/101129/#comment2455> Please add a blank after the , kpresenter/part/KPrViewModeSlidesSorter.cpp <http://git.reviewboard.kde.org/r/101129/#comment2456> Please add a blank after the , kpresenter/part/KPrViewModeSlidesSorter.cpp <http://git.reviewboard.kde.org/r/101129/#comment2458> As m_zoom is a member variable the int needs to be removed otherwise you are shadowing the variable. - Thorsten On April 20, 2011, 2:52 a.m., Paul Mendez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101129/ > ----------------------------------------------------------- > > (Updated April 20, 2011, 2:52 a.m.) > > > Review request for Calligra. > > > Summary > ------- > > Add save of zoom factor and mode for normal view and slides sorter. Also > eliminate page and width options of zooming in Slides Sorter (these options > doesn't make much sense in that view). > Finally I eliminate the white space around the slide because it make the > slide lost focus when zooming and when I restore the previous zoom value (the > zoom function center the white space too), in addition the white space make > too big the default size of shapes when inserted. > (I ask in calligra channel, and the white space was added to have an area for > insert things, maybe it could be added in a better way). > > > Diffs > ----- > > kpresenter/part/KPrView.h 80b67520be60500ecf11df0827f1d4ee8eeab4e7 > kpresenter/part/KPrView.cpp aef56d9fcb919a314f0095629bea1927916f8059 > kpresenter/part/KPrViewModeSlidesSorter.h > ec5f6691072115eb7818f1eb224f6a03fe9bf13c > kpresenter/part/KPrViewModeSlidesSorter.cpp > b4fee3c755aebf65e9bff0cef5516776533ec4a6 > libs/kopageapp/KoPAView.h 5585364c4bf0503e312a302b91f762653aaf98bb > libs/kopageapp/KoPAView.cpp c53edecb309098949f66e9d50238bc60f26f9bee > > Diff: http://git.reviewboard.kde.org/r/101129/diff > > > Testing > ------- > > build and run > > > Thanks, > > Paul > >
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
