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