Skip to content

WIP Use subsurfaces for screenshots UI#155

Draft
ids1024 wants to merge 4 commits intomasterfrom
screenshot-subsurfaces
Draft

WIP Use subsurfaces for screenshots UI#155
ids1024 wants to merge 4 commits intomasterfrom
screenshot-subsurfaces

Conversation

@ids1024
Copy link
Member

@ids1024 ids1024 commented Apr 3, 2025

Based on top of #153.

This seems to help a lot with performance. It needs to be updated to use the right images for background subsurfaces (probably by sharing code with cosmic-bg), but it also has some issue where sometimes one output doesn't show the background subsurface? That seems to be avoided by disabling the needs_sorting test in iced_winit, but that can lead to protocol errors reordering subsurfaces...

@ids1024 ids1024 force-pushed the screenshot-subsurfaces branch from 83e9811 to ac77249 Compare April 7, 2025 16:35
@ids1024 ids1024 force-pushed the screenshot-subsurfaces branch from ac77249 to 8d402fe Compare May 15, 2025 19:57
@ids1024 ids1024 force-pushed the screenshot-subsurfaces branch from 8d402fe to 1f85977 Compare June 12, 2025 19:50
@ids1024
Copy link
Member Author

ids1024 commented Jun 12, 2025

There have been some subsurfaces fixes in iced since this, and as far as I can tell it doesn't have issues anymore. Need to update it to use the right background.

Should also be able to use an `image::ImageBuffer` wrapping shm memory
without a copy when no transform is applied. But this doesn't do that.
@ids1024 ids1024 force-pushed the screenshot-subsurfaces branch from 1f85977 to d558af7 Compare October 6, 2025 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant