D21613: Format focal length as milli meter
    Stefan BrĂ¼ns 
    noreply at phabricator.kde.org
       
    Thu Jun  6 00:23:59 BST 2019
    
    
  
bruns added inline comments.
INLINE COMMENTS
> propertyinfotest.cpp:144
>          { Property::AspectRatio,            true,  1.77778,                 QStringLiteral("1.78:1")},
> +        { Property::PhotoFocalLength,       true,  2.0,                     QStringLiteral("2 mm")},
>      };
For small frame cameras (compact cameras, smartphones, ...) the real focal length is often in the range of only a few millimeters, please add a test case which checks e.g. "2.4 mm" is formatted as such.
REPOSITORY
  R286 KFileMetaData
BRANCH
  focal_length
REVISION DETAIL
  https://phabricator.kde.org/D21613
To: astippich, bruns, ngraham
Cc: kde-frameworks-devel, #baloo, LeGast00n, domson, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190605/6440f817/attachment.html>
    
    
More information about the Kde-frameworks-devel
mailing list