2008/10/8 Aaron J. Seigo <[EMAIL PROTECTED]>: >> On the other hand, the shell does not react well to configuration >> changes. It is necessary to restart plasma when a monitor is >> added/removed. > > there is code to handle this, actually. but we've never had the signals to > actually test it and so it's sort of rotted out. see PlasmaApp::adjustSize in > workspace/plasma/shells/desktop/plasmaapp.cpp ..
Yes, it seems signals indeed are part the problem here. adjustSize is called in response to a signal in QDesktopWidget, which is, as Aike pointed out before, pre randr-1.2 stuff. The issue is that the argument to the signal is not really what is expected by adjustSize. I guess the right thing to do would be to listen to kephal signals instead of QDesktopWidget ones? How would communication work in that case? dbus? Anyway, I'll re-apply the workaround patch I sent before, to see if it is still useful (it reconfigured all the screens instead of reconfiguring only one when adjustSize was called). If it works, it might be a good idea to include it until a kephal is moved out of playground. Cheers, g > > you should find this: > > //TODO: we have a screen that has changed, but no view. > // perhaps we should make one. > > ;) > >> Aike told me Seli is working on that. > > Seli is working on plasma? or just working on making kwin not interfere with > screen change announcements? > > -- > 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@kde.org > https://mail.kde.org/mailman/listinfo/plasma-devel > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
