Review Request: Add KMetaDataWidget, KMetaDataModel and KMetaDataConfigurationDialog
Peter Penz
peter.penz at gmx.at
Sun Mar 14 01:49:49 GMT 2010
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3277/
-----------------------------------------------------------
Review request for kdelibs, Sebastian Trueg, David Faure, and Aurélien Gâteau.
Summary
-------
The patch adds KMetaDataWidget, KMetaDataModel and KMetaDataConfigurationDialog as public classes to kdelibs/kfile. The KMetaDataWidget allows an application in an easy way to show meta data of a file (or several files). The widget also allows to change meta data like tags, comments and rating (see http://enzosworld.gmxhome.de/temp/metadatawidget.png or attached screenshot). KMetaDataConfigurationDialog allows to configure which meta tags should be hidden/shown. The classes also work without Nepomuk (and show only very basic meta data like size, permissions, ...). It is possible for applications to add custom meta data if wanted (including widgets to manipulate this meta data).
The classes have been used by Dolphin internally until now and have originally been written by Sebastian Trüg. After the request from Tom Albers and Oliver Heidbüchel to integrate the widget also in Mailody/Okular I've adjusted the classes to get them ready for a kdelibs-integration. I'd also like to to adjust KPropertiesDialog later to use this widget.
I'd ask mainly to look at the files kfile/kmetadatawidget.h, kfile/kmetadatamodel.h and kfile/kmetadataconfigurationdialog.h, the other APIs are internal.
Thanks!
Diffs
-----
trunk/KDE/kdelibs/kfile/kcommentwidget_p.h PRE-CREATION
trunk/KDE/kdelibs/kfile/config-nepomuk.h.cmake PRE-CREATION
trunk/KDE/kdelibs/kfile/kcommentwidget.cpp PRE-CREATION
trunk/KDE/kdelibs/kfile/CMakeLists.txt 1102931
trunk/KDE/kdelibs/kfile/kedittagsdialog.cpp PRE-CREATION
trunk/KDE/kdelibs/kfile/kedittagsdialog_p.h PRE-CREATION
trunk/KDE/kdelibs/kfile/kloadmetadatathread.cpp PRE-CREATION
trunk/KDE/kdelibs/kfile/kloadmetadatathread_p.h PRE-CREATION
trunk/KDE/kdelibs/kfile/kmetadataconfigurationdialog.h PRE-CREATION
trunk/KDE/kdelibs/kfile/kmetadataconfigurationdialog.cpp PRE-CREATION
trunk/KDE/kdelibs/kfile/kmetadatamodel.h PRE-CREATION
trunk/KDE/kdelibs/kfile/kmetadatamodel.cpp PRE-CREATION
trunk/KDE/kdelibs/kfile/kmetadatawidget.h PRE-CREATION
trunk/KDE/kdelibs/kfile/kmetadatawidget.cpp PRE-CREATION
trunk/KDE/kdelibs/kfile/knfotranslator.cpp PRE-CREATION
trunk/KDE/kdelibs/kfile/knfotranslator_p.h PRE-CREATION
trunk/KDE/kdelibs/kfile/ktaggingwidget.cpp PRE-CREATION
trunk/KDE/kdelibs/kfile/ktaggingwidget_p.h PRE-CREATION
trunk/KDE/kdelibs/nepomuk/core/ui/CMakeLists.txt 1102931
trunk/KDE/kdelibs/nepomuk/core/ui/nepomukmassupdatejob.h 1102931
trunk/KDE/kdelibs/nepomuk/core/ui/nepomukmassupdatejob.cpp 1102931
Diff: http://reviewboard.kde.org/r/3277/diff
Testing
-------
Tested in Dolphin. An early version has been tested also in Mailody and Okular.
Screenshots
-----------
KMetaDataWidget
http://reviewboard.kde.org/r/3277/s/330/
Thanks,
Peter
More information about the kde-core-devel
mailing list