plasma apps and screen information

[Guillaume Pothier]

Ok, here comes a new version of the patch. I think I took into account
all your comments: the new methods in Corona have a default
implementation, I removed the ViewerCorona class, moved popupPosition
to Corona, removed the convenience methods from View and Applet. The
only remaining problem, I think, is the ToolTipManager stuff. I think
if it remains a singleton the only way to access the corona from it is
to have the corona register itself when created. Ok to commit?
g

2008/10/27 Aaron J. Seigo <aseigo at kde.org>:
> On Monday 27 October 2008, Guillaume Pothier wrote:
>> 2008/10/27 Aaron J. Seigo <aseigo at kde.org>:
>> >> The latter should probably return a QRegion instead, according to what
>> >> you said about toolbox placement.
>> >
>> > yes, it should be a QRegion.
>>
>> ok, I'll use QRegion in the API, but for now the region will be
>> rectangular and the toolbox positioning code will rely on the region's
>> bounding box. Later I'll see how to improve that. Is that ok?
>
> absolutely. it's a good pragmatic "one step at a time" approach
>
>
>> >> - Added a ViewerCorona class to plasmoidviewer, as Plasma::Corona is
>> >> now has pure virtual methods.
>> >
>> > shouldn't be necessary if they aren't pure virtual. that you had to do
>> > this work was probably a good hint something could be better =)
>>
>> I think plasmoidviewer needs its own corona anyway so as to report the
>> actual window size instead of a meaningless value?
>
> no; it should just set the sceneRect to be equal to the window size. i'm
> pretty sure it already does that, actually.
>
> --
> 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
>
>
> _______________________________________________
> Plasma-devel mailing list
> Plasma-devel at kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
>
>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: plasma-corona.diff
Url: http://mail.kde.org/pipermail/plasma-devel/attachments/20081028/244db3a9/attachment-0001.bat