Review Request 122413: Add paintedWidth and paintedHeight to QPixmapItem

Luca Beltrame lbeltrame at kde.org
Tue Feb 3 22:22:12 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122413/
-----------------------------------------------------------

(Updated Feb. 3, 2015, 10:22 p.m.)


Review request for KDE Frameworks and Plasma.


Changes
-------

Return 0 for both methods if m_pixmap is null (not already set).


Repository: kdeclarative


Description
-------

Like the corresponding properties in QML's Image, this patch introduces paintedWidth and paintedHeight to QPixmapItem, to get information on the size of the pixmap that's actually shown (depending on fillMode), as compared to its actual size.

This brings QPixmapItem closer to the mode of usage of Image.

CHANGELOG: New properties paintedWidth and paintedHeight added to QPixmapItem


Diffs (updated)
-----

  src/qmlcontrols/kquickcontrolsaddons/qpixmapitem.h 485d6f7 
  src/qmlcontrols/kquickcontrolsaddons/qpixmapitem.cpp 6c9e3a2 

Diff: https://git.reviewboard.kde.org/r/122413/diff/


Testing
-------

Builds. 


Thanks,

Luca Beltrame

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150203/d22d649b/attachment.html>


More information about the Kde-frameworks-devel mailing list