Review Request 126822: AppletQuickItem: Don't try to set initial size bigger than parent size

[David Rosca]

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

(Updated Jan. 21, 2016, 2:41 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Changes
-------

Submitted with commit f688c904a81c72d077580ab942b874f398fbd7d4 by David Rosca to branch master.


Bugs: 358200
    http://bugs.kde.org/show_bug.cgi?id=358200


Repository: plasma-framework


Description
-------

Trying to set initial size bigger than parent size only results to item being resized down to fit in parent later.

This fixes issue in this case:
Applet sets its initial size with width and height properties, this size makes the applet switch to full representation. This applet is in panel and this initial size doesn't fit in panel. So at first, the applet gets resized to this size, this triggers switch to full representation from compactRepresentationCheck. Applet is then resized to fit in panel, making it to switch to compact representation. The initial size (from width, height properties) is now lost.


Diffs
-----

  src/plasmaquick/appletquickitem.cpp 995ec10 

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


Testing
-------

Bug fixed.


Thanks,

David Rosca

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160121/8936d02d/attachment.html>