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

(Updated March 10, 2011, 8:36 p.m.)


Review request for Calligra.


Changes
-------

I fixed the comments, sorry for the extra stuff in the same patch (just the 
idea come to my mind when working in other stuff, and I wanted to try it). I 
agree that zooming could be improved, it was just a first approach and I could 
make some changes after finishing with the current patch. I agree with store 
and restore the zoom level for this view mode, that seems the best solution. 
Regarding with max and min levels, the max level probably result usefulness, I 
could add some zooming limitations and left only the zoom range that could 
result useful for the view mode purposes. 


Summary
-------

Correct how slides sorter updates when add/delete/copy/paste a slide. Add some 
copy/paste extra context menu, and the supr key for delete slides in slides 
sorter.

Any suggestion for add Contrl C/V/X support?


Diffs (updated)
-----

  kpresenter/part/CMakeLists.txt 61c9848 
  kpresenter/part/KPrSlidesSorterDocumentModel.h e69de29 
  kpresenter/part/KPrSlidesSorterDocumentModel.cpp e69de29 
  kpresenter/part/KPrViewModeSlidesSorter.h 743fbe9 
  kpresenter/part/KPrViewModeSlidesSorter.cpp 24e6c1e 
  libs/kopageapp/KoPADocumentStructureDocker.cpp 08fb66a 
  libs/kopageapp/KoPAView.h 55a6b7d 

Diff: http://git.reviewboard.kde.org/r/100672/diff


Testing
-------


Thanks,

Paul

_______________________________________________
calligra-devel mailing list
calligra-devel@kde.org
https://mail.kde.org/mailman/listinfo/calligra-devel

Reply via email to