Review Request 119860: hints to disable background in dialogs

[Marco Martin]

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

(Updated Aug. 21, 2014, 1:55 p.m.)


Review request for Plasma and Pier Luigi Fiorini.


Repository: plasma-framework


Description
-------

add the possibility to not have backgrounds in dialogs.
one ting i don't like that much is the fact that it pretty much forces to either having perfect rectangular windows or to have composite for correct looks, but that will be probably less important.
One thing that should be paid attention is that with no background is not supported the kwin-side shadow, as well no blur as well, since we don't have info for the shape, tough if needed in the future blur with just the full rectangular shape may be added as a new flag in backgroundhints, those possibilities of expansion is one reason it's an enum not a bool.

The use case for it is for custom dialogs in Maui, and since Dialog already manages the million edge cases of sizing and positioning that we had to fix, the least painful route i think is to use them with the background disabled.
the framesvgitem is still there, but not having a svg loaded shouldn't take memory for images neither in heap nor gpu (probably would be good as well to set the flag itemhasnocontents on framesvgitem if imagepath is invalid, to be really sure it will never even try to upload any texture)


Diffs (updated)
-----

  src/plasmaquick/dialog.h 27b3b5a 
  src/plasmaquick/dialog.cpp 64fb4a9 

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


Testing
-------


Thanks,

Marco Martin

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