Review Request: Fix the layer docker of karbon to work again after part/document split

Thorsten Zachmann t.zachmann at zagge.de
Sun Aug 26 08:05:45 BST 2012


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

Review request for Calligra and Jan Hambrecht.


Description
-------

When the part/document split happened the layer docker got broken as not all was ported correctly.

This patch uses the KoCanvasObserverBase to see that the document has changed and acts on that instead of using the karbon part.


This addresses bug 305259.
    http://bugs.kde.org/show_bug.cgi?id=305259


Diffs
-----

  karbon/ui/KarbonView.cpp 620626e 
  karbon/ui/dockers/KarbonLayerDocker.h df349d4 
  karbon/ui/dockers/KarbonLayerDocker.cpp 6831bf9 
  karbon/ui/widgets/KarbonCanvas.h 790fcdb 
  karbon/ui/widgets/KarbonCanvas.cpp 6371f5a 

Diff: http://git.reviewboard.kde.org/r/106210/diff/


Testing
-------

Opened a document with layers. The layers are displayed again and it is possible to manipulate them.


Thanks,

Thorsten Zachmann

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20120826/511048df/attachment.htm>


More information about the calligra-devel mailing list