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