plasma apps and screen information

Aaron J. Seigo aseigo at kde.org
Fri Oct 24 02:12:52 CEST 2008


On Thursday 23 October 2008, Guillaume Pothier wrote:
> 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 =)
>
> It ended up a much bigger task than I thought... adding the methods to
> corona and making use of them was rather simple, but I also changed
> the naming from "screen" to "viewport" in Containment, View, etc...
> lots of renaming. And when I think I'm done, I realize that View
> extends QAbstractScrollArea through QGraphicsView, and that class has
> a viewport() method that returns a QWidget*... It seems the overloaded
> method is still accessible by explicitly qualifying it
> ("QGraphicsView::viewport()"), but is it ok to overload that method?

hm... so viewport is a bad name. i completely forgot that we'd need that 
method in a scroll area subclass (View) ... well, let's leave it at screen 
then, and just allow the Corona to say what a "screen" means.

-- 
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/20081023/a85ba1c1/attachment.sig 


More information about the Plasma-devel mailing list