[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-core-devel
Subject: Re: Review Request: Add KMetaDataWidget,
From: "Peter Penz" <peter.penz () gmx ! at>
Date: 2010-03-16 7:42:50
Message-ID: 20100316074250.21474.92059 () localhost
[Download RAW message or body]
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3277/#review4514
-----------------------------------------------------------
trunk/KDE/kdelibs/kfile/kmetadatawidget.h
<http://reviewboard.kde.org/r/3277/#comment3984>
Ah, of course I meant:
void setVisible(const KUrl& metaDataUri, bool visible);
instead of
void setVisible(const KUrl& metaDataUri);
- Peter
On 2010-03-14 01:49:48, Peter Penz wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/3277/
> -----------------------------------------------------------
>
> (Updated 2010-03-14 01:49:48)
>
>
> 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
>
>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic