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 spritesheet import logic #813

Merged
merged 1 commit into from
Mar 16, 2024

Conversation

Hades948
Copy link
Contributor

Add logic to the SpritesheetImportPanel that automatically sets the default columns/rows if the file width is divisible by the file height.

This new logic does not apply when opening the spritesheet from a SpritesheetResource. The user could, for example, have a 64x16 spritesheet that they have already set to be one row and two columns. The new logic would change the number of columns to 4 when opening the spritesheet from a SpritesheetResource. This is likely not what the user would want, so the new logic is not applied in this case.

Add logic to the SpritesheetImportPanel that automatically sets the
default columns/rows if the file width is divisible by the file height.

This new logic does not apply when opening the spritesheet from a
SpritesheetResource. The user could, for example, have a 64x16
spritesheet that they have already set to be one row and two columns.
The new logic would change the number of columns to 4 when opening
the spritesheet from a SpritesheetResource. This is likely not what
the user would want, so the new logic is not applied in this case.
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Amazing, your first Pull Request in the LITIENGINE repository! You have earned the rank of "contributor" - let us know your nickname in the Forum and on Discord so we can reward you with the new role :).

@Hades948 Hades948 marked this pull request as ready for review March 10, 2024 19:40
Copy link
Member

@nightm4re94 nightm4re94 left a comment

Choose a reason for hiding this comment

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

Thanks, everything looks in order :)

@nightm4re94 nightm4re94 merged commit e79a922 into gurkenlabs:main Mar 16, 2024
1 check 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.

None yet

2 participants