Review Request: Better TagWidget in libnepomuk
Sebastian Trueg
trueg at kde.org
Mon Apr 19 12:28:07 BST 2010
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3669/
-----------------------------------------------------------
Review request for kdelibs and Peter Penz.
Summary
-------
Nepomuk::TagWidget has been there forever but due to its being rather unintuitive and ugly (despite my efforts of making it fancy back in the day) it is not used at all. Since we have the class in kdelibs and a tagging widget is a typical thing to use I made another attempt at creating a generic one that apps ca use. It basically shows a list of checkboxes of the N most often used tags and has a button to show all tags. This button will bring up the exact dialog used in Dolphin and Gwenview (the one from KIO). So far the code cannot be shared since we cannot link libnepomuk against kio. But maybe we could do it the other way around and move the tag edit dialog into libnepomuk as a public class.
Anyway, this review request is about the new TagWidget and whether it has a chance to be used. I added a simple test app that allows to toy around with the widget.
Diffs
-----
trunk/KDE/kdelibs/includes/CMakeLists.txt 1114717
trunk/KDE/kdelibs/includes/Nepomuk/TagWidget PRE-CREATION
trunk/KDE/kdelibs/nepomuk/core/CMakeLists.txt 1114717
trunk/KDE/kdelibs/nepomuk/core/ui/CMakeLists.txt 1114717
trunk/KDE/kdelibs/nepomuk/core/ui/kautoscrollarea.h 1114717
trunk/KDE/kdelibs/nepomuk/core/ui/kautoscrollarea.cpp 1114717
trunk/KDE/kdelibs/nepomuk/core/ui/kblocklayout.h 1114717
trunk/KDE/kdelibs/nepomuk/core/ui/kblocklayout.cpp 1114717
trunk/KDE/kdelibs/nepomuk/core/ui/kedittagsdialog.cpp PRE-CREATION
trunk/KDE/kdelibs/nepomuk/core/ui/kedittagsdialog_p.h PRE-CREATION
trunk/KDE/kdelibs/nepomuk/core/ui/kmetadatatagwidget.h 1114717
trunk/KDE/kdelibs/nepomuk/core/ui/kmetadatatagwidget.cpp 1114717
trunk/KDE/kdelibs/nepomuk/core/ui/tagcheckbox.h PRE-CREATION
trunk/KDE/kdelibs/nepomuk/core/ui/tagcheckbox.cpp PRE-CREATION
trunk/KDE/kdelibs/nepomuk/core/ui/tagwidget.h PRE-CREATION
trunk/KDE/kdelibs/nepomuk/core/ui/tagwidget.cpp PRE-CREATION
trunk/KDE/kdelibs/nepomuk/core/ui/test/CMakeLists.txt 1114717
trunk/KDE/kdelibs/nepomuk/core/ui/test/tagwidgetapp.cpp PRE-CREATION
trunk/KDE/kdelibs/nepomuk/core/ui/test/tagwidgettest.h PRE-CREATION
trunk/KDE/kdelibs/nepomuk/core/ui/test/tagwidgettest.cpp PRE-CREATION
Diff: http://reviewboard.kde.org/r/3669/diff
Testing
-------
Thanks,
Sebastian
More information about the kde-core-devel
mailing list