Status of multi-monitor support

Aaron J. Seigo aseigo at kde.org
Mon Oct 13 20:48:50 CEST 2008


On Friday 10 October 2008, Guillaume Pothier wrote:
> http://pastebin.ca/1225015 (kdebase)
> http://pastebin.ca/1225017 (playground/base)

things that jump out at me:

Cosistency with the rest of KDE libs:

* the "kephal" namespace shoulde be "Kephal"
* instance() should be self()

Questions;

* why is "kephal::Screens::instance()->screens().size()" used sometimes and 
"kdephal::ScreenUtils::numScreens()" used instead?

* why does DesktopCorona now have a m_numScreens member instead of just 
tracking whatever kephal says it is as the moment?

* i suppose that PlasmaApp::screenMoved would need to move the associated view 
so it is still on screen?

* with the code to manage the containments in DesktopCorona, we can probably 
just remove the containment size related code in DesktopView. PlasmaApp can 
manage just the View geometries and DesktopCorona just the containment 
geometries? it's a little muddled right now due to how things evolved over 
time.

-- 
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/20081013/091a4dbc/attachment.sig 


More information about the Plasma-devel mailing list