Use expandedGeometry to calculate the thumbnail geometry
Review Request #106336 - Created Sept. 5, 2012 and submitted
summarized, see bug. nb, that there's either a "bug" in oxygen shadows or the deco-but-not-the-deco code, at least setting the oxygen shadow size to 40 gets me a padding of 80px on either side (this is regardless of this patch), just noticed during testing. However, this leads to smaller windows (than would likely be required) and does (visually) not happen with new shadows. I'll have a look unless Hugo says "no, wait: the UI config is doubled by intent)
Shadow size is set as the maximum of inactive shadow and active glow read from config. as far as I could check (adding a printout before returning the metrics for LM_OuterPadding* (oxygenclient, line 399 or so), this is the right value that is used, no times 2 there ... Or am I not checking the right thing ?
Concerning "new shadows" (assuming we are talking about passing shadows to kwin as X properties), there is a (now rather old) remote branch named "origin/hpereira/oxygen-shadows" that implements just that but I was told by Fredrick (and sort of verified) that it was not a good idea, performance wise, to make the switch ... no clue whether this has changed or not. I guess merging will be a bit more difficult now since the branch is smthing like 6 month old, but that can still be achieved.
Other than that, patch seems to work almost ok. Case when I could make it fail: maximize a window vertically only (or horizontally) It appears without shadow. Now move it around (without resizing), it still appears unclipped. Things are fixed at next resize of the window (to something smaller)