Review Request: Stage: save zoom factor for normal view and slides sorter between sesions

Paul Mendez paulestebanms at gmail.com
Wed Apr 20 03:52:35 BST 2011


-----------------------------------------------------------
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.


Changes
-------

The zoom is stored in temporal variables when zoom change and saved to config in the destructor of the view, also I added a Center method in KoPaView to recenter the slides each time the view is loaded again or zoom changes.


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 (updated)
-----

  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

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


More information about the calligra-devel mailing list