Review Request 116915: Fix Bug 305618

Wenchao Li manchiu.lee.9 at gmail.com
Thu Mar 20 11:42:40 GMT 2014


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

(Updated March 20, 2014, 11:42 a.m.)


Review request for Calligra.


Changes
-------

It's my first contribution. So I made some changes to follow the code style.


Summary (updated)
-----------------

Fix Bug 305618


Repository: calligra


Description (updated)
-------

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/KPrPageApplicationData.cpp bfda40b 
  stage/part/tools/animationtool/KPrPageEffectDocker.h e7a5277 
  stage/part/tools/animationtool/KPrPageEffectDocker.cpp 836d687 

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


Testing
-------


Thanks,

Wenchao Li

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20140320/0e3aaf1e/attachment.htm>


More information about the calligra-devel mailing list