-
Notifications
You must be signed in to change notification settings - Fork 81
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use the window-pixmaps provided by the compositor of Xwm4 #53
Comments
Hmm, I'm not sure how this would work, since Skippy-XD is foremost desktop independent. And even if xfwm4 provided such functionality, I'm not sure that other WMs would. Which would imply adding xfwm4-specific code to Skippy-XD... |
Right, I guess I presumed it was tied more to Xfce, but I guess you're right. |
Yeah, a compile-time option would make sense... |
There is also a subtle change if this is done that may fix other behavioral issues. Specifically when changing workspaces the windows are unmapped leading to only the Icon preview and not the windows contents. From a quick look at the feature, they are saving the pixmaps for the unmapped windows that exist in other workspaces. Thus if skippy-xd implemented this API to fetch pixmaps, it would also contain backing images for all windows across all workspaces. I was recently looking to hack xfwm4 to do exactly that, since I am working on trying to create a slide visual effect when changing workspaces and thought perhaps I could cheat and hack skippy-xd to provide a workspace change animation. First I'd need to get backing images of the windows off-screen and thought xfwm4 would have to be modified before that was possible, what a nice discovery! I would be up for trying to implement this API for xfwm4 if no one else is. |
Xfwm4's compositor recently gained an API to retrieve the window pixmap (as can be seen here: http://git.xfce.org/xfce/xfwm4/commit/?id=f6d5236b887c062117ee504358e54e63a98e3a54)
I would imagine this could simplify skippy's code quite a bit, care to take a peek at that?
The text was updated successfully, but these errors were encountered: