Review Request: Stage: Enhace behaviour of copy/paste slides in slide sorter

[Paul Mendez]

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

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