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

Add save button to twix #1665

Merged
merged 8 commits into from
Mar 9, 2025

Conversation

Franziska-Sophie
Copy link
Contributor

Why? What?

Adds a "Save" button to twix in the image pannel that saves the camera image as PNG/JPEG file. The images are then stored in /tmp/twix.

How to Test

Open the image pannel in twix and click on the save button (with and without the JPEG box checked). Check in /tmp/twix if the image was saved.

@oleflb oleflb self-assigned this Mar 5, 2025
Copy link
Contributor

@oleflb oleflb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice addition! Still need to test but looks good to me

@oleflb
Copy link
Contributor

oleflb commented Mar 5, 2025

Oh and you need to increase the minor version in twix 😉 You can find it in the twix Cargo.toml :)

@Franziska-Sophie Franziska-Sophie requested a review from oleflb March 7, 2025 16:58
Copy link
Contributor

@oleflb oleflb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice :)

@oleflb
Copy link
Contributor

oleflb commented Mar 8, 2025

Please enable merge when ready to merge the PR.

@schmidma
Copy link
Member

schmidma commented Mar 9, 2025

@oleflb all comments have to be resolved to have this PR merged

@oleflb
Copy link
Contributor

oleflb commented Mar 9, 2025

resolved now. still waiting for merge when ready @Franziska-Sophie

@Franziska-Sophie Franziska-Sophie added this pull request to the merge queue Mar 9, 2025
Merged via the queue into HULKs:main with commit e1d84b4 Mar 9, 2025
32 checks passed
@Franziska-Sophie Franziska-Sophie deleted the twix-save-button branch March 9, 2025 21:18
ThagonDuarte pushed a commit to ThagonDuarte/hulk that referenced this pull request Mar 10, 2025
* add save button to twix for saving camera image as png/jpeg

* fix imports

* update twix version to 0.9.3

* moved implementation of saving a jpeg image to an impl block of JpegImage

* wip

* removed some comments

* format 'jpeg.rs'
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants