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, 7:54 a.m.)


Review request for Calligra.


Changes
-------

I follow the coding style and fix all small stuff now. And others code remains unchanged.


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/tools/animationtool/KPrPageEffectDocker.h e7a5277 
  stage/part/commands/KPrPageEffectSetCommand.h ec95f07 
  stage/part/commands/KPrPageEffectSetCommand.cpp f44953f 

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


Testing
-------

Tested manually.


Thanks,

Wenchao Li

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