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 Feb. 17, 2011, 8:55 p.m.)


Review request for Calligra.


Changes
-------

You are right, the problem were because of the previous patch, so I updated the new patch.

Regarding support for Ctrl CVX, I need some guidance about the standard method for implement these keys, I look for these in the keyPress event manager of the widgets but I didn't find implementations of these keys in other parts of calligra (on keypressevent method), maybe I'm missing some standard of calligra/kde? (I have worked with Qt in the past, but I a bit new with kde specific stuff).

I am going to review how optimize the update of slides.
thanks

Paul


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/KPrViewModeSlidesSorter.h 743fbe9 
  kpresenter/part/KPrViewModeSlidesSorter.cpp 24e6c1e 
  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/20110217/d6f2791d/attachment.htm>