plasma apps and screen information

Guillaume Pothier gpothier at gmail.com
Fri Oct 24 17:08:09 CEST 2008


2008/10/23 Aaron J. Seigo <aseigo at kde.org>:
> On Wednesday 22 October 2008, Guillaume Pothier wrote:
>> Just tell me if you want me to go ahead.
>
> please do =)

Ok, here comes the patch. In summary:
- Added three methods to Corona:
   int numScreens()
   QRect screenGeometry(int)
   QRect availableScreenGeometry(int)
The latter should probably return a QRegion instead, according to what
you said about toolbox placement.
- QDesktopWidget is not used anymore in libplasma, entirely replaced
by calls to the new Corona methods.
- Added Corona* parameter to Plasma::popupPosition
- Added a corona() method to Plasma::Applet. This is needed for calls
to Plasma::popupPosition.
- Also added a corona() method to Plasma::View for consistency's sake.
- Corona is friends with ToolTipManager, as ToolTipManager needs to
call Plasma::popupPosition and thus needs a Corona. Thus, Corona
registers itself with the ToolTipManager. I think this is the ugliest
part of the patch, please tell me if there is a better solution.
- Added a ViewerCorona class to plasmoidviewer, as Plasma::Corona is
now has pure virtual methods.

Ok to commit?
g
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: plasma-corona.diff
Url: http://mail.kde.org/pipermail/plasma-devel/attachments/20081024/06c2a17a/attachment-0001.bat 


More information about the Plasma-devel mailing list