Review Request 120653: Move KoRuler, KoRulerController and KoFontComboBox from komain to kowidgets
Friedrich W. H. Kossebau
kossebau at kde.org
Tue Oct 21 19:44:56 BST 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120653/
-----------------------------------------------------------
(Updated Oct. 21, 2014, 6:44 p.m.)
Status
------
This change has been marked as submitted.
Review request for Calligra.
Repository: calligra
Description
-------
Wigdets (and even more QWidget-based ones) make most sense if all collected in kowidgets, and not only some.
kowidgets already contains many widgets (which know about classes from kotext and flake).
So I propose to move KoRuler, KoRulerController and KoFontComboBox from komain to kowidgets. That way QML-based apps that use komain do not need to care about these classes. And koreport no longer needs to link to komain, but can "just" go with kowidgets and its deps.
Diffs
-----
libs/widgets/KoRulerController.cpp PRE-CREATION
libs/widgets/KoRulerController_p.h PRE-CREATION
libs/widgets/KoRuler_p.h PRE-CREATION
libs/widgets/KoRuler.cpp PRE-CREATION
libs/widgets/KoRulerController.h PRE-CREATION
libs/koreport/CMakeLists.txt 3f648b7
libs/main/CMakeLists.txt 2affede
libs/main/KoFontComboBox.h 47d1550
libs/main/KoRuler.h 2c4875d
libs/main/KoRuler.cpp 6f49a1e
libs/main/KoRulerController.h 5543da7
libs/main/KoRulerController.cpp 9b433f4
libs/main/KoRulerController_p.h 1779972
libs/main/KoRuler_p.h 7c63e0a
libs/widgets/CMakeLists.txt 2268054
libs/widgets/KoFontComboBox.h PRE-CREATION
libs/widgets/KoRuler.h PRE-CREATION
Diff: https://git.reviewboard.kde.org/r/120653/diff/
Testing
-------
Builds fine here.
Thanks,
Friedrich W. H. Kossebau
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20141021/8d19200b/attachment.htm>
More information about the calligra-devel
mailing list