[PATCH] fix plasma and Qt monitor-hotplug issues

Aike J Sommer dev at aikesommer.name
Fri Mar 28 13:16:15 CET 2008


Attached are 2 patches to fix issues with monitor hotplug.
The patch to qt-copy fixes:
- screenResized not beeing emitted on some changes
- screenResized always reporting screen 0 to have changed

This is a little hacky... Qt uses Xinerama to query for screens, so listening 
to XRandR 1.2 events doesnt really work... So this patch will cause 
ResizeEvents beeing triggered even if it might not be necessary to make 
QDesktopWidget recalc the geometry!
Where should i go with that from here?? :-)

The patch to kdebase basically only calls view->show() after creating it... 
The rest just reduces some iterations over all screens, which are not needed 
anymore!

One thing left: Should views be destroyed if a monitor is disconnected? And if 
so i guess using screenOwnerChanged or a (new) signal containmentRemoved 
wouldnt be right, since containments should stay associated with the 
screen... Maybe in DesktopCorona::adjustSize()?

:-)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: qt-copy-patch-01.diff
Type: text/x-diff
Size: 1712 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20080328/d6c73ad8/attachment.bin 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kdebase-patch-02.diff
Type: text/x-diff
Size: 4584 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20080328/d6c73ad8/attachment-0001.bin 


More information about the Panel-devel mailing list