Skip to content

main: implement saveScreenshot() writing bmp from SDL window buffer#10

Merged
NeroBurner merged 1 commit into
mainfrom
screenshots
Mar 8, 2022
Merged

main: implement saveScreenshot() writing bmp from SDL window buffer#10
NeroBurner merged 1 commit into
mainfrom
screenshots

Conversation

@NeroBurner

Copy link
Copy Markdown
Collaborator

When pressing i create a screenshot of the current InfiniTime screen
by dumping it to a timestamped bmp file.

Fixes: #5

@NeroBurner

Copy link
Copy Markdown
Collaborator Author

@JF002 could you test if this PR creating bmp screenshots is working for you

@JF002

JF002 commented Mar 8, 2022

Copy link
Copy Markdown
Contributor

It works for me! Thanks! 💯 I tried both BMP and PNG. I think PNG is better as the files are much smaller.
If you want to add a few more functionalities, here are 2 ideas:

  • Copy the image in the clipboard
  • Give a different name (add a file number or a timestamp to the file name, for example).

NeroBurner added a commit to NeroBurner/InfiniTime that referenced this pull request Mar 8, 2022
InfiniSim PR InfiniTimeOrg/InfiniSim#10 adds
support for screenshots in png format using a `libpng` submodule. This
new submodule must be added to the lv_sim workflow as well.
When pressing `i` create a screenshot of the current InfiniTime screen
by dumping it to a timestamped png or bmp file like
`InfiniSim_2022-03-08_203421.png`.

Add a new configure option `WITH_PNG` with default `ON`. When switched
to `OFF` the screenshots will be created in `bmp` format.

Fixes: #5
@NeroBurner

Copy link
Copy Markdown
Collaborator Author

updated with PNG support by adding a new configure option WITH_PNG with default ON. When switched
to OFF the screenshots will be created in bmp format. Otherwise in png format

This also adds a new submodule libpng for the png format support

@NeroBurner NeroBurner merged commit 2e81595 into main Mar 8, 2022
@NeroBurner NeroBurner deleted the screenshots branch March 8, 2022 20:45
NeroBurner added a commit to NeroBurner/InfiniTime that referenced this pull request Mar 8, 2022
InfiniSim PR InfiniTimeOrg/InfiniSim#10 adds
support for screenshots in png/bmp format using `libpng` submodule. This
new submodule must be added to the lv_sim workflow as well.
NeroBurner added a commit to NeroBurner/InfiniTime that referenced this pull request Mar 8, 2022
InfiniSim PR InfiniTimeOrg/InfiniSim#10 adds
support for screenshots in png/bmp format using `libpng` submodule.
This new submodule must be added to the lv_sim workflow as well.
JF002 pushed a commit to InfiniTimeOrg/InfiniTime that referenced this pull request Mar 8, 2022
InfiniSim PR InfiniTimeOrg/InfiniSim#10 adds
support for screenshots in png/bmp format using `libpng` submodule.
This new submodule must be added to the lv_sim workflow as well.
grad0s pushed a commit to aramcon-badge/InfiniTime that referenced this pull request Jun 25, 2022
InfiniSim PR InfiniTimeOrg/InfiniSim#10 adds
support for screenshots in png/bmp format using `libpng` submodule.
This new submodule must be added to the lv_sim workflow as well.
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.

Make screenshot and write image

2 participants