Frameworks/Epics/KDEUI Crumble
< Frameworks | Epics
KDEUI Crumble
This page is the battle plan to finally get rid of KDEUI in kdelibs/frameworks. It lists the header name of all the classes in kdeui, they are sorted in sections per new destination for them.
Each section tries to also provide simple indications of the contraints to follow during the move (mostly about dependencies). If a class is not fitting one of the constraints, it is part of the job to first reduce the dependencies of the class to fit the contraints prior to the move. If you are too blocked to carry on, please add a note for the class.
The sections are roughly sorted by order of difficulty. The classes in the first sections should be reasonably easy to move, while the ones in the last sections are more challenging.
Tip: you can use http://lxr.kde.org/ to find out where in KDE the class is used.
KDE4Support
Points to verify before moving the class in KDE4Support:
- Make sure the class is used only in kde4support itself or kde4attic but not anywhere else in kdelibs
- Don't forget to change the export in the moved header
- Add API doc in the header file about it being deprecated and indicate how to port
- Look at the bug reports for the component
| Status | Description | Notes | Contact |
|---|---|---|---|
| DONE | ./fonts/kfontdialog.h deprecated for QFontDialog | Review 110351 | <annma@kde.org> |
| IN PROGRESS | ./fonts/kfontcombobox.h deprecated for QFontComboBox | Review 110456 | <davidedmundson@kde.org> |
| IN PROGRESS | ./dialogs/kinputdialog.h deprecated for QInputDialog | David Gil | <davidgiloliva@gmail.com> |
| TODO | ./dialogs/kdeprintdialog.h (should be done by Qt, kcups* goes with it) | ... | <null@kde.org> |
| DONE | ./widgets/keditlistbox.h deprecated for KEditListWidget | Review 110464 | <annma@kde.org> |
| TODO | ./widgets/kstringvalidator.h only the zombie kfilemetainfowidget uses it | ... | <null@kde.org> |
| DONE | ./widgets/ktabbar.h deprecated for QTabBar | Review 110460 | <alex.merry@kdemail.net> |
| DONE | ./widgets/ktabwidget.h deprecated for QTabWidget | ... | <sebas@kde.org> |
| IN PROGRESS | ./widgets/kbuttongroup.h Use real QGroupBox and QButtonGroup instances please, kthxbye | Review 110478 | <annma@kde.org> |
| IN PROGRESS | ./widgets/knuminput.h Use real QSpinBox | Review 110418 Waiting Monday 3rd June before pushing so people have time to update their Qt5 | <annma@kde.org> |
| DONE | ./widgets/ktextbrowser.h deprecated for QTextBrowser | Review 110437 | <annma@kde.org> |
| DONE | ./util/kpixmapcache.h deprecated for KImageCache or KSharedDataCache | Review 110500 | <alex.merry@kdemail.net> |
| DONE | ./util/kaccelgen.h | Review 110540 | <alex.merry@kdemail.net> |
KDE4Attic
Points to verify before moving the class in KDE4Attic [[1]]:
- Make sure the class is not used anywhere in kdelibs anymore
| Status | Description | Notes | Contact |
|---|---|---|---|
| DONE | ./widgets/kdatetimeedit.h | https://git.reviewboard.kde.org/r/110415/ | <wojtask9.kde@gmail.com> |
| TODO | ./widgets/kdatepicker.h | used in security/crypto/kdatetimedlg.cpp | <null@kde.org> |
| TODO | ./widgets/kmultitabbar.h (used by kate, digikam, kexi --- but could use a cleanup, see e.g. KDEV3ICON...) | ... | <null@kde.org> |
| DONE | ./widgets/kdatecombobox.h | http://git.reviewboard.kde.org/r/110447/ | <wojtask9.kde@gmail.com> |
| DONE | ./widgets/ktimezonewidget.h | http://git.reviewboard.kde.org/r/110451/ | <wojtask9.kde@gmail.com> |
| DONE | ./widgets/kdatetimewidget.h | https://git.reviewboard.kde.org/r/110457/ | <wojtask9.kde@gmail.com> |
| TODO | ./widgets/kcharselect.h Can soon go into kwidgetsaddons (needs clear button in QLineEdit) | ... | <null@kde.org> |
| DONE | ./widgets/kdatewidget.h | Review 110622 | <annma@kde.org> |
| DONE | ./widgets/ktimecombobox.h | Review 110477 | <annma@kde.org> |
| TODO | ./widgets/klanguagebutton.h | used in kdeui/dialogs/kswitchlanguagedialog_p.cpp | <null@kde.org> |
| TODO | ./widgets/kdatetable.h | See kdatepicker | <null@kde.org> |
| TODO | ./util/kpassivepopupmessagehandler.h only user is kde-runtime/knotify. Move it there? | ... | <null@kde.org> |
| TODO | ./shortcuts/kgesture.h use #if 0 for the code in xmlgui (KShortcutsEditor) that uses KGesture (it's already hidden from the user anyway) | ... | <null@kde.org> |
| TODO | ./shortcuts/kgesturemap.h needs polish before it can be resurrected | ... | <null@kde.org> |
KGuiAddons
Points to verify before moving a class in KGuiAddons:
- It depends only on QtGui or QtCore, eventually QX11Info
| Status | Description | Notes | Contact |
|---|---|---|---|
| IN PROGRESS | ./util/kmodifierkeyinfo.h | ... | <scarpino@kde.org> |
| DONE | ./fonts/kfontutils.h | 110646 | <kurt.hindenburg@gmail.com> |
| TODO | ./util/kimagecache.h but but but... KsharedDataCache is in kcoreaddons :-/ (tough problem) | ... | <null@kde.org> |
KWidgetsAddons
Points to verify before moving a class in KWidgetsAddons:
- It depends only on QtWidgets, QtGui or QtCore
- It can depend on other classes in KWidgetsAddons, but that should be kept minimal
| Status | Description | Notes | Contact |
|---|---|---|---|
| DONE | ./fonts/kfontrequester.h | ... | <davidedmundson@kde.org> |
| TODO | ./dialogs/kassistantdialog.h | ... | <null@kde.org> |
| IN PROGRESS | ./dialogs/knewpassworddialog.h | ... | <davidedmundson@kde.org> |
| DONE | ./dialogs/kpassworddialog.h | https://git.reviewboard.kde.org/r/110643/ | <davidedmundson@kde.org> |
| IN PROGRESS | ./dialogs/kpixmapregionselectordialog.h | https://git.reviewboard.kde.org/r/110847/ | <aleixpol@kde.org> |
| IN PROGRESS | ./widgets/kpixmapregionselectorwidget.h | https://git.reviewboard.kde.org/r/110847/ | <aleixpol@kde.org> |
| DONE | ./widgets/ktitlewidget.h | commit 44da8fea214dc41243c6aed9caa32e0cda7d3027 | <ervin@kde.org> |
| DONE | ./widgets/kratingwidget.h | https://git.reviewboard.kde.org/r/110826/ | <aleixpol@kde.org> |
| DONE | ./widgets/kratingpainter.h | https://git.reviewboard.kde.org/r/110826/ | <aleixpol@kde.org> |
| TODO | ./widgets/kmessagewidget.h (Needs to get rid of K* using fixed default colors, Qstyle :styleHint and Qstyle :standardPixmap) | ... | <null@kde.org> |
| DONE | ./widgets/keditlistwidget.h | https://git.reviewboard.kde.org/r/110825/ | <aleixpol@kde.org> |
| DONE | ./widgets/kactionselector.h | https://git.reviewboard.kde.org/r/110820/ | <aleixpol@kde.org> |
| IN PROGRESS | ./util/kpassivepopup.h | https://git.reviewboard.kde.org/r/110815/ | <aleixpol@kde.org> |
| DONE | ./util/kcursor.h | https://git.reviewboard.kde.org/r/110827/ | <aleixpol@kde.org> |
| IN PROGRESS | ./paged/kpagewidget.h | Waiting for https://git.reviewboard.kde.org/r/110833/ | <aleixpol@kde.org> |
| IN PROGRESS | ./paged/kpagedialog.h | Waiting for https://git.reviewboard.kde.org/r/110833/ | <aleixpol@kde.org> |
| IN PROGRESS | ./paged/kpageview.h | Waiting for https://git.reviewboard.kde.org/r/110833/ | <aleixpol@kde.org> |
| IN PROGRESS | ./paged/kpagemodel.h | Waiting for https://git.reviewboard.kde.org/r/110833/ | <aleixpol@kde.org> |
| IN PROGRESS | ./paged/kpagewidgetmodel.h | Waiting for https://git.reviewboard.kde.org/r/110833/ | <aleixpol@kde.org> |
| TODO | ./shortcuts/kacceleratormanager.h the list of translated texts from standard actions will have to be set by the first KStandardAction being constructed | ... | <null@kde.org> |
| TODO | ./colors/kcolorbutton.h minor duplication of KcolorMimeData though | ... | <null@kde.org> |
| DONE | ./colors/kcolorcombo.h | http://git.reviewboard.kde.org/r/110838/ | <aleixpol@kde.org> |
| TODO | ./actions/kactionmenu.h | ... | <null@kde.org> |
| TODO | ./actions/ktoolbarlabelaction.h | ... | <null@kde.org> |
| TODO | ./actions/ktoolbarpopupaction.h | ... | <null@kde.org> |
| DONE | ./actions/ktoolbarspaceraction.h | Review 110674 | <aleixpol@kde.org>
|
KWindowSystem
Points to verify before moving a class in KWindowSystem:
- It depends only on QtWidgets, QtGui or QtCore
- It can depend on other classes in KWindowSystem, but that should be kept minimal
| Status | Description | Notes | Contact |
|---|---|---|---|
| TODO | ./util/kkeyserver_x11.h | ... | <null@kde.org> |
| TODO | ./util/kxerrorhandler.h | ... | <null@kde.org> |
| TODO | ./util/kkeyserver_win.h | ... | <null@kde.org> |
| TODO | ./util/kkeyserver_mac.h | ... | <null@kde.org> |
| TODO | ./util/kkeyserver.h | ... | <null@kde.org> |
KWidgets
Points to verify before moving a class in KWidgets:
- It doesn't use anything aimed at KCompletion (e.g. KLineEdit or KComboBox)
- It doesn't use anything aimed at ItemViews
- It doesn't use anything from kdecore
- Using things from tier1 is fine
- Using things from tier2 is forbidden
| Status | Description | Notes | Contact |
|---|---|---|---|
| TODO | ./dialogs/kaboutapplicationdialog.h (probably difficult) | ... | <null@kde.org> |
| TODO | ./dialogs/kbugreport.h | ... | <null@kde.org> |
| TODO | ./dialogs/ktip.h | ... | <null@kde.org> |
| TODO | ./widgets/khelpmenu.h | ... | <null@kde.org> |
ItemViews
Points to verify before moving a class in ItemViews:
- It doesn't use anything aimed at KCompletion (e.g. KLineEdit or KComboBox)
- It doesn't use ki18n
- It doesn't use anything from kdecore
- It doesn't use anything from kwidgets
- Using things from tier1 is fine
| Status | Description | Notes | Contact |
|---|---|---|---|
| DONE | ./itemviews/kcategorizedview.h | Review: 110825, 110825 | <wojtask9.kde@gmail.com> |
| DONE | ./itemviews/klistwidgetsearchline.h (port away from KlineEdit and i18n) | Review: 110825, 110825 | <wojtask9.kde@gmail.com> |
| DONE | ./itemviews/ktreewidgetsearchline.h (port away from KlineEdit and i18n) | Review: 110825, 110825 | <wojtask9.kde@gmail.com> |
| DONE | ./itemviews/kwidgetitemdelegate.h | Review: 110825, 110825 | <wojtask9.kde@gmail.com> |
| DONE | ./itemviews/kfilterproxysearchline.h (port away from KlineEdit and i18n) | Review: 110825, 110825 | <wojtask9.kde@gmail.com> |
| DONE | ./itemviews/kcategorydrawer.h | Review: 110825, 110825 | <wojtask9.kde@gmail.com> |
| IN PROGRESS | ./itemviews/kcategorizedsortfilterproxymodel.h (need porting away from KstringHandler) | Review: 110825, 110825 | <wojtask9.kde@gmail.com> |
KCompletion
Points to verify before moving a class in KCompletion:
- It doesn't use anything from kwidgets
- It doesn't use anything from kdecore
- It doesn't use anything from kdeui
- Using things from tier1 is fine
| Status | Description | Notes | Contact |
|---|---|---|---|
| TODO | ./widgets/khistorycombobox.h | ... | <null@kde.org> |
| TODO | ./widgets/kcombobox.h | ... | <null@kde.org> |
| TODO | ./widgets/klineedit.h | ... | <null@kde.org> |
| TODO | ./widgets/kcompletionbox.h | ... | <null@kde.org> |
| DONE | ./util/kcompletion.h move ksortablelist.h with it | ... | <ervin@kde.org> |
KConfigWidgets
Points to verify before moving a class in KConfigWidgets:
- It uses only classes from kconfigcore, kconfigui or kwidgetsaddons
| Status | Description | Notes | Contact |
|---|---|---|---|
| IN PROGRESS | ./dialogs/kconfigdialogmanager.h | old unit test crashes on load! | <kde@davidedmundson.co.uk> |
| TODO | ./dialogs/kconfigdialog.h | ... | <null@kde.org> |
| TODO | ./widgets/kcmodule.h | ... | <null@kde.org> |
| TODO | ./itemviews/kviewstatesaver.h | ... | <null@kde.org> |
| TODO | ./itemviews/kviewstatemaintainer.h | ... | <null@kde.org> |
KCrash
Important: At time of writing the KCrash framework doesn't exist yet in staging, the person picking the first move task here would have to do that.
Points to verify before moving a class in KCrash:
- It uses only classes from kcoreaddons or kwindowsystem
| Status | Description | Notes | Contact |
|---|---|---|---|
| TODO | ./util/kcrash.h | ... | <null@kde.org> |
KIconThemes
Points to verify before moving a class in KIconThemes:
- It uses only classes from tier1 or tier2
- It uses nothing from kdeui or kdecore
| Status | Description | Notes | Contact |
|---|---|---|---|
| DONE | ./util/kpixmapsequencewidget.h | https://git.reviewboard.kde.org/r/110733/ | <wojtask9.kde@gmail.com> |
| DONE | ./util/kpixmapsequenceoverlaypainter.h | https://git.reviewboard.kde.org/r/110733/ | <wojtask9.kde@gmail.com> |
| DONE | ./util/kpixmapsequence.h | https://git.reviewboard.kde.org/r/110733/ | <wojtask9.kde@gmail.com> |
| TODO | ./icons/kiconbutton.h | ... | <null@kde.org> |
| TODO | ./icons/kicondialog.h | ... | <null@kde.org> |
KJobWidgets
Points to verify before moving a class in KJobWidgets:
- It uses only classes from Qt, kcoreaddons or kwidgetsaddons
| Status | Description | Notes | Contact |
|---|---|---|---|
| TODO | ./jobs/kwidgetjobtracker.h | ... | <null@kde.org> |
| TODO | ./jobs/kabstractwidgetjobtracker.h | ... | <null@kde.org> |
| TODO | ./jobs/kdialogjobuidelegate.h | ... | <null@kde.org> |
| TODO | ./jobs/kstatusbarjobtracker.h | ... | <null@kde.org> |
KNotifications
Points to verify before moving a class in KNotifications:
- It uses only classes from KWindowSystem, QtWidgets, QtDBus
| Status | Description | Notes | Contact |
|---|---|---|---|
| IN PROGRESS | ./notifications/kstatusnotifieritem.h | ... | <mart@kde.org> |
| IN PROGRESS | ./notifications/knotificationrestrictions.h | ... | <mart@kde.org> |
KWallet
Important: At time of writing the KWallet framework doesn't exist yet in staging, the person picking the first move task here would have to do that.
Points to verify before moving a class in KWallet:
- It uses only classes from kconfigcore, kconfigui or kwindowsystem
| Status | Description | Notes | Contact |
|---|---|---|---|
| TODO | ./util/kwallet.h | ... | <null@kde.org> |
XmlGui
Points to verify before moving a class in XmlGui:
- It uses only classes from tier1 or tier2
- It uses nothing from kdeui or kdecore
| Status | Description | Notes | Contact |
|---|---|---|---|
| TODO | ./dialogs/kshortcutseditor.h | ... | <null@kde.org> |
| TODO | ./dialogs/kshortcutsdialog.h | ... | <null@kde.org> |
| TODO | ./dialogs/kedittoolbar.h | ... | <null@kde.org> |
| TODO | ./widgets/kshortcutwidget.h | ... | <null@kde.org> |
| TODO | ./widgets/kkeysequencewidget.h | ... | <null@kde.org> |
| TODO | ./util/kundoactions.h | ... | <null@kde.org> |
| TODO | ./kdepackages.h (goes with kbugreport) | ... | <null@kde.org> |
| TODO | ./xmlgui/kxmlguiclient.h | ... | <null@kde.org> |
| TODO | ./xmlgui/kxmlguiwindow.h | ... | <null@kde.org> |
| TODO | ./xmlgui/kxmlguifactory.h | ... | <null@kde.org> |
| TODO | ./xmlgui/kmainwindow.h | ... | <null@kde.org> |
| TODO | ./xmlgui/kxmlguibuilder.h | ... | <null@kde.org> |
| TODO | ./xmlgui/ktoolbar.h | ... | <null@kde.org> |
| TODO | ./shortcuts/kcheckaccelerators.h | ... | <null@kde.org> |
| TODO | ./actions/kactioncollection.h | ... | <null@kde.org> |
| TODO | ./actions/ktoggletoolbaraction.h | ... | <null@kde.org> |
| TODO | ./actions/kactioncategory.h | ... | <null@kde.org> |
KTextWidgets
Important: At time of writing the KTextWidgets framework doesn't exist yet in staging, the person picking the first move task here would have to do that.
Points to verify before moving a class in KTextWidgets:
- It uses only classes from tier1 or tier2
- It uses nothing from kdeui or kdecore
| Status | Description | Notes | Contact |
|---|---|---|---|
| TODO | ./findreplace/kfinddialog.h | ... | <null@kde.org> |
| TODO | ./findreplace/kreplacedialog.h | ... | <null@kde.org> |
| TODO | ./findreplace/kfind.h | ... | <null@kde.org> |
| TODO | ./findreplace/kreplace.h | ... | <null@kde.org> |
| TODO | ./widgets/krichtextwidget.h | ... | <null@kde.org> |
| TODO | ./widgets/nestedlisthelper.h (goes with krichtextedit) | ... | <null@kde.org> |
| TODO | ./widgets/ktextedit.h | ... | <null@kde.org> |
| TODO | ./widgets/krichtextedit.h | ... | <null@kde.org> |
| TODO | ./dialogs/klinkdialog.h | It's not exported, but needed for KRichTextEdit | <null@kde.org>
|