RFC: Drop KScreen dependency from PlasmaShell
Review Request #125451 - Created Sept. 29, 2015 and submitted
|Aleix Pol Gonzalez|
Now that everything's in place in Qt, we can do that.
It basically removes all the translation layer between KScreen and QScreen.
NOTE: This can't be merged until Qt 5.6 is a dependency.
Hands-on testing, it's been working quite fluid and I couldn't find a crash. (There could be crashes, I just didn't find them :D)
+1 as well. KScreen should only be used to configure the screen setup, not to query available screens.
Thanks a lot for this work, Aleix, especially getting our needs upstreamed into Qt.
Why is this connect in showEvent?
this won't work.
X struts are relative to the global work area (i.e all the screens)
if you have a small laptop with a high res external screen, when you unplug the external screen you'll need to update the struts of a panel on the smaller screen.
This signal wouldn't be emitted in that case. I'm not sure what the right signal is.
I've just applied this patch and it fixed all my multi-screen/laptop/docking issues! You literally saved my relationship! (Gf switch her laptop to plasma 5 and was constantly cursing)