D19216: Karbon: Enable multi page capability
Dag Andersen
noreply at phabricator.kde.org
Fri Feb 22 10:00:49 GMT 2019
danders created this revision.
danders added a reviewer: anthonyfieroni.
Herald added a project: Calligra: 3.0.
Herald added a subscriber: Calligra-Devel-list.
danders requested review of this revision.
REVISION SUMMARY
Since odg spec supports multiple pages, I feel karbon also needs to support it.
Ported to use pageapp classes.
Karbon has config group "Interface" that is not precent in other apps.
Some of the options have been disabled atm. Imho they should be
harmonized with other apps and/or moved to View menu.
A lot of code was duplicated between pageapp and karbon
and has been removed from karbon:
- Save/load
- Layers docker and all layer operations
- Grid, guides, rulers and zoom
- Event handlers
- Printing
- Show page margins has been moved to pageapp
In general, import/export needs review to determine how to
handle multiple pages when e.g. exporting to a format that
does not support pages.
Known bugs:
- "Separate paths" command:
- Execute command, the shape disappears.
- Undo crashes. Note: Afaics this code is not touched so probably a libs bug.
- Snap to grid does not work
TEST PLAN
I am not an avid user of karbon, so would be nice if some that where could test.
Also, do not have all types of different format docs for filter testing.
Some that work are svg, jpg and karbon files.
REPOSITORY
R8 Calligra
BRANCH
karbon_multipage_danders
REVISION DETAIL
https://phabricator.kde.org/D19216
AFFECTED FILES
filters/karbon/CMakeLists.txt
filters/karbon/image/CMakeLists.txt
filters/karbon/image/ImageExport.cpp
filters/karbon/svg/CMakeLists.txt
filters/karbon/svg/SvgExport.cpp
filters/karbon/svg/SvgImport.cpp
filters/karbon/wmf/CMakeLists.txt
filters/karbon/wmf/WmfExport.cpp
karbon/CMakeLists.txt
karbon/data/karbon.rc
karbon/ui/CMakeLists.txt
karbon/ui/KarbonDocument.cpp
karbon/ui/KarbonDocument.h
karbon/ui/KarbonDocumentMergeCommand.cpp
karbon/ui/KarbonDocumentMergeCommand.h
karbon/ui/KarbonPart.cpp
karbon/ui/KarbonPart.h
karbon/ui/KarbonPrintJob.cpp
karbon/ui/KarbonPrintJob.h
karbon/ui/KarbonView.cpp
karbon/ui/KarbonView.h
karbon/ui/ProxyView.cpp
karbon/ui/ProxyView.h
karbon/ui/dockers/KarbonLayerDocker.cpp
karbon/ui/dockers/KarbonLayerDocker.h
karbon/ui/dockers/KarbonLayerModel.cpp
karbon/ui/dockers/KarbonLayerModel.h
karbon/ui/dockers/KarbonLayerSortingModel.cpp
karbon/ui/dockers/KarbonLayerSortingModel.h
karbon/ui/widgets/KarbonCanvas.cpp
karbon/ui/widgets/KarbonCanvas.h
karbon/ui/widgets/KarbonConfigInterfacePage.cpp
karbon/ui/widgets/KarbonSmallStylePreview.h
libs/pageapp/KoPAView.cpp
libs/pageapp/KoPAView.h
To: danders, anthonyfieroni
Cc: Calligra-Devel-list, dcaliste, cochise, vandenoever
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20190222/421892b8/attachment.htm>
More information about the calligra-devel
mailing list