D7696: Add "Width" and "Height" separately to Additional Information -> Images

David Edmundson noreply at phabricator.kde.org
Tue Sep 5 08:47:46 BST 2017


davidedmundson added inline comments.

INLINE COMMENTS

> kbaloorolesprovider.cpp:147
>          { "height",        "imageSize" },
> +        { "width",         "width" },
> +        { "height",        "height" },

You're setting an entry that ultimately gets stored in a QHash, and duplicating a key used previously.

Can you double check that you've not broken image size.

REPOSITORY
  R318 Dolphin

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

To: wael.ch, #dolphin, #kde_applications
Cc: davidedmundson, ngraham, navarromorales, nicolasfella, firef, andrebarros, emmanuelp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20170905/725d0359/attachment.htm>


More information about the kfm-devel mailing list