-
Notifications
You must be signed in to change notification settings - Fork 373
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
Central GpuReadback handling for re_viewer, experimental space view screenshots #1717
Conversation
f7f6fc6
to
e72f502
Compare
Can't run api_demo_rs anymore:
|
Can confirm the screenshots are successfully taken and save to disk on Linux, both x11 and Wayland. The copy-to-clipboard thing only works when using the x11 backend though; I think that's because we're trying to use the x11 clipboard APIs even when running on the Wayland backend:
Note the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pleasantly straightforward 👍
I personally wouldn't mind an extra option to disable saving to disk, I hate having stuff getting written to my disk 😬
strange, api demo compiles just fine for me! |
ah I think you haven't been on the latest commit where I fixed web build? |
Just for you @teh-cmc ;)
(yes the ui looks ugly, don't care atm, it's experimental anyways) |
Screen.Recording.2023-03-27.at.15.27.07.mov
Created the embryo of handling readback data in a generalized & robust fashion across the viewer. Using this to implement screenshot functionality on 2d/3d space views - since it's limited to those & doesn't include egui rendered labels, I put this behind a feature flag.
Note that this is force-disabled on the Web.
Checklist