Grouping Desktop moved to kdereview

[Aaron J. Seigo]

On July 24, 2010, Giulio Camuffo wrote:
> Actually it seems like 4.5 doesn't install anymore the kephal headers. Is
> this something wanted?

kephal is currntly for use by kdebase-workspace only.

i see you moved to using QDesktopWidget ... while that probably mostly works, 
it assumes that this is being used in a full screen workspace. that's an 
assumption that is likely to hold up, but not necessarily.

for Containments, init() is called prior to restore(), which is where settings 
such as default sizes are set up. this is actually the opposite to Applets, 
where restore() is called first.

since this is a pure Containment (won't be used as an Applet), you can rely on 
that bit of knowledge. (if that wasn't the case, you could check for 
isContainment() in init() to know :)

anyways .. in init(), you have access to the Corona. and the Corona offers 
screenGeometry(int screen), which allows for app/platform specific 
implementations of what that means. for Plasma Desktop, that is implemented 
using Kephal.

so you could instead just use corona()->screenGeometry() in init(). :)

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Development Frameworks
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20100726/63dd49b3/attachment.sig>