bruns added inline comments.

INLINE COMMENTS

> propertyinfo.cpp:185
> +            d->name = QStringLiteral("equipmentManufacturer");
> +            d->displayName = i18nc("@label EXIF", "Equipment Manufacturer");
>              d->valueType = QVariant::String;

Isn't `Manufacturer` sufficient for the label? Same for `Model`.

REPOSITORY
  R286 KFileMetaData

BRANCH
  euqipment_properties

REVISION DETAIL
  https://phabricator.kde.org/D21519

To: astippich, ngraham, bruns
Cc: kde-frameworks-devel, #baloo, gennad, domson, ashaposhnikov, michaelh, 
astippich, spoorun, ngraham, bruns, abrahams

Reply via email to