Review Request 116915: Fix Bug 305618

[Wenchao Li]

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

(Updated March 23, 2014, 2:47 a.m.)


Review request for Calligra.


Changes
-------

Correct the mistakes. Realize in another way with a parent command. Only one redo or undo operation.


Repository: calligra


Description
-------

The patch tries to add a new function about "add this slide animation for all slides". After the checkbox is checked, the presentation will use the same slide transition for all slides.
The function "slotApplyToAllSlides()" is defined in file KPrPageEffectDocker.cpp which implements the task.


Diffs (updated)
-----

  stage/part/tools/animationtool/KPrPageEffectDocker.cpp 836d687 
  stage/part/commands/KPrPageEffectSetCommand.h ec95f07 
  stage/part/commands/KPrPageEffectSetCommand.cpp f44953f 
  stage/part/tools/animationtool/KPrPageEffectDocker.h e7a5277 

Diff: https://git.reviewboard.kde.org/r/116915/diff/


Testing (updated)
-------

Tested manually.


Thanks,

Wenchao Li

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