4.2 feature targets

[Aaron J. Seigo]

On Monday 20 October 2008, Guillaume Pothier wrote:
> * Containment::setScreen: used to change the geometry of the
> containment when the screen is changed. We could avoid that if we say
> the views are in charge of resizing the containments.

.. leading to duplicated code in Views. i'd rather not.

see my email "plasma apps and screen information" for an alternate solution.

> * ContainmentPrivate::positionToolbox, used to correctly position the
> toolbox. It seems that here we could rely on the geometry of the
> containment itself, as it is supposed to be the same size as the
> screen, and the toolbox is positioned relative to the containment.

we still need the available area, however. that's really the whole point of 
that method, really.

still, this should probably be gotten from plasma as we can do a much better 
job than the current WM can i think. in fact, we could change it to returning 
a region or set of rects that are out of bounds, allowing us to avoid the 
"panel is at the top, but doesn't cover the right corner and yet the toolbox 
is still too low"

> * ContainmentPrivate::positionContainment. Used to find the maximum
> width/height of all screens. Maybe we could use instead the max
> width/height of all containments, for the same reason as above.

you need to svn up =)

> * Containment::effectiveScreenPos. There it seems we cannot use the
> containment's geometry as we need the actual screen position.

yes, but see my screen positioning email

> * Containment::destroy. Only the number of screens is used.

ditto

> * popupPosition in plasma.cpp: Needs the actual screen position
>
> * PopupAppletPrivate::updateDialogPosition. Needs the actual screen
> position.

i'm not sure if these two can be addressed in the same manner, as i just 
haven't looked into it ... but .. maybe?

-- 
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 Software

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20081020/90efbfb3/attachment.sig