<table><tr><td style="">danders created this revision.<br />danders added a reviewer: anthonyfieroni.<br />danders added a project: Calligra: 3.0.<br />danders requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D19327">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Since odg spec supports multiple pages, I feel karbon also needs to support it.</p>

<p>Ported to use pageapp classes.</p>

<p>Karbon has config group "Interface" that is not precent in other apps.<br />
Some of the options have been disabled atm. Imho they should be<br />
harmonized with other apps and/or moved to View menu.</p>

<p>A lot of code was duplicated between pageapp and karbon<br />
and has been removed from karbon:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Save/load</li>
<li class="remarkup-list-item">Layers docker and all layer operations</li>
<li class="remarkup-list-item">Grid, guides, rulers and zoom</li>
<li class="remarkup-list-item">Event handlers</li>
<li class="remarkup-list-item">Printing</li>
<li class="remarkup-list-item">Show page margins has been moved to pageapp</li>
</ul>

<p>In general, import/export needs review to determine how to<br />
handle multiple pages when e.g. exporting to a format that<br />
does not support pages.</p>

<p>Known bugs:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">"Separate paths" command:<ul class="remarkup-list">
<li class="remarkup-list-item">Execute command, the shape disappears.</li>
<li class="remarkup-list-item">Undo crashes. Note: Afaics this code is not touched so probably a libs bug.</li>
</ul></li>
</ul>

<ul class="remarkup-list">
<li class="remarkup-list-item">Snap to grid does not work</li>
</ul>

<ul class="remarkup-list">
<li class="remarkup-list-item">Number of recent files does not work properly</li>
</ul></div></div><br /><div><strong>TEST PLAN</strong><div><p>I am not an avid user of karbon, so would be nice if some that where could test.<br />
Also, do not have all types of different format docs for filter testing.<br />
Some that work are pdf, svg, jpg and karbon files.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R8 Calligra</div></div></div><br /><div><strong>BRANCH</strong><div><div>karbon_multipage_danders</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19327">https://phabricator.kde.org/D19327</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>filters/karbon/CMakeLists.txt<br />
filters/karbon/image/CMakeLists.txt<br />
filters/karbon/image/ImageExport.cpp<br />
filters/karbon/svg/CMakeLists.txt<br />
filters/karbon/svg/SvgExport.cpp<br />
filters/karbon/svg/SvgImport.cpp<br />
filters/karbon/wmf/CMakeLists.txt<br />
filters/karbon/wmf/WmfExport.cpp<br />
karbon/CMakeLists.txt<br />
karbon/data/karbon.rc<br />
karbon/ui/CMakeLists.txt<br />
karbon/ui/KarbonDocument.cpp<br />
karbon/ui/KarbonDocument.h<br />
karbon/ui/KarbonDocumentMergeCommand.cpp<br />
karbon/ui/KarbonDocumentMergeCommand.h<br />
karbon/ui/KarbonPart.cpp<br />
karbon/ui/KarbonPart.h<br />
karbon/ui/KarbonPrintJob.cpp<br />
karbon/ui/KarbonPrintJob.h<br />
karbon/ui/KarbonView.cpp<br />
karbon/ui/KarbonView.h<br />
karbon/ui/ProxyView.cpp<br />
karbon/ui/ProxyView.h<br />
karbon/ui/dockers/KarbonLayerDocker.cpp<br />
karbon/ui/dockers/KarbonLayerDocker.h<br />
karbon/ui/dockers/KarbonLayerModel.cpp<br />
karbon/ui/dockers/KarbonLayerModel.h<br />
karbon/ui/dockers/KarbonLayerSortingModel.cpp<br />
karbon/ui/dockers/KarbonLayerSortingModel.h<br />
karbon/ui/widgets/KarbonCanvas.cpp<br />
karbon/ui/widgets/KarbonCanvas.h<br />
karbon/ui/widgets/KarbonConfigInterfacePage.cpp<br />
karbon/ui/widgets/KarbonSmallStylePreview.h<br />
libs/pageapp/KoPAView.cpp<br />
libs/pageapp/KoPAView.h</div></div></div><br /><div><strong>To: </strong>danders, anthonyfieroni<br /><strong>Cc: </strong>Calligra-Devel-list, dcaliste, cochise, vandenoever<br /></div>