Skip to content

Conversation

@Platane
Copy link
Owner

@Platane Platane commented Nov 18, 2025

No description provided.

@Platane Platane force-pushed the add-options branch 5 times, most recently from c447e73 to 5637eb9 Compare November 18, 2025 19:54
Copy link

@TheGiraffe3 TheGiraffe3 left a comment

Choose a reason for hiding this comment

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

A few wording suggestions.


Having a transparent background seems to have a quite negative impact on the file size and duration. It should be avoided by default.

Transparency is currently broken for `gif-encoder-2`

Choose a reason for hiding this comment

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

Suggested change
Transparency is currently broken for `gif-encoder-2`
Transparency is currently broken for `gif-encoder-2`.


Transparency is currently broken for `gif-encoder-2`

**Interpolaction**

Choose a reason for hiding this comment

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

Suggested change
**Interpolaction**
**Interpolation**


**Interpolaction**

It is possible to interpolate betwen two snake step for a smoother movement. This again has a huge impact on the file size and should be avoided.

Choose a reason for hiding this comment

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

Suggested change
It is possible to interpolate betwen two snake step for a smoother movement. This again has a huge impact on the file size and should be avoided.
It is possible to interpolate between two snake steps for a smoother movement. This again has a huge impact on the file size and should be avoided.

@Platane Platane marked this pull request as ready for review November 19, 2025 18:44
@Platane Platane merged commit 4364e44 into main Nov 19, 2025
4 checks passed
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.

3 participants