Review Request: Extract the Cursor Theme configuration from the Input (Mouse Config) KCM
Darío Andrés
andresbajotierra at gmail.com
Mon Oct 12 01:49:12 BST 2009
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1832/
-----------------------------------------------------------
(Updated 2009-10-12 00:49:12.369570)
Review request for kdelibs.
Changes
-------
Small update
Summary (updated)
-------
This is a simple extraction/split up of the cursor theme configuration from the Input KCM. I copied(moved) most of the code.
I only added the KCM main source file with all the copyrights from the Input KCM "mouse.h" and "mouse.cpp" files.
The next steps is remove all the files that this module includes from the Input module (that would be another review request/patch)
This split up is required in our plan of SystemSettings reorganization. In case that the "big reorganization" is not done, we are going to move this new module to "Appearance" as most of the users thinks that is the proper place to find the mouse cursor theme config.
Regards
--
The .desktop file still points to the "keyboard-and-mouse" category. We should redefine that place after the discussion.
While the diff here doesn't show the file moves properly, I have a full copy in my local build (with full svn history)
Oh, I forgot to append the Messages.sh file here, but it is in my local copy too.
Diffs
-----
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/CMakeLists.txt 1033579
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/CMakeLists.txt PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/core/bitmaps.h PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/core/themepage.h PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/core/themepage.cpp PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/cursor_large.bdf PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/cursor_large_black.pcf.gz UNKNOWN
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/cursor_large_white.pcf.gz UNKNOWN
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/cursor_small_white.pcf.gz UNKNOWN
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/cursortheme.h PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/cursortheme.cpp PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/cursortheme.desktop PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/kapplymousetheme.cpp PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/mouse_cursor_theme.upd PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/xcursor/bitmaps.h PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/xcursor/cursortheme.h PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/xcursor/cursortheme.cpp PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/xcursor/itemdelegate.h PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/xcursor/itemdelegate.cpp PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/xcursor/legacytheme.h PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/xcursor/legacytheme.cpp PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/xcursor/previewwidget.h PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/xcursor/previewwidget.cpp PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/xcursor/sortproxymodel.h PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/xcursor/sortproxymodel.cpp PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/xcursor/thememodel.h PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/xcursor/thememodel.cpp PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/xcursor/themepage.h PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/xcursor/themepage.cpp PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/xcursor/themepage.ui PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/xcursor/xcursortheme.h PRE-CREATION
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/kcontrol/cursortheme/xcursor/xcursortheme.cpp PRE-CREATION
Diff: http://reviewboard.kde.org/r/1832/diff
Testing
-------
The module is compiled, and it works properly. The helper tool to apply the theme is working too.
Thanks,
Darío
More information about the kde-core-devel
mailing list