Skip to content
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

Deal with built-in screenshot code #29

Open
akx opened this issue Oct 7, 2019 · 1 comment · May be fixed by #42
Open

Deal with built-in screenshot code #29

akx opened this issue Oct 7, 2019 · 1 comment · May be fixed by #42
Assignees
Labels

Comments

@akx
Copy link
Contributor

akx commented Oct 7, 2019

The built-in screenshot code writes bespoke RIX files.

We should either

  • make it write PNGs or something via SDL_image, or
  • get rid of built-in screenshotting instead and leave it to the OS.

I'd vote for getting rid of it.

@akx akx added type:question Further information is requested type:cleanup area:io area:graphics labels Oct 7, 2019
@akx akx self-assigned this Oct 7, 2019
@veikkos
Copy link
Member

veikkos commented Oct 7, 2019

Yea, removal is fine.

akx added a commit that referenced this issue Feb 13, 2023
@akx akx linked a pull request Feb 13, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging a pull request may close this issue.

2 participants