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

Maniacs Feature - Remove scaling restrictions for pictures #3249

Merged
merged 1 commit into from
Sep 2, 2024

Conversation

ToolMan2k
Copy link
Contributor

From Maniac Patch version 240423:

[RPG_RT.exe]
Support for different magnification rates for special drawing of pictures (rotation, waveform, angle).
  • Added check if sprite's width/height is at zero. If so, skip the draw routine (helps avoid a glitchy sprite at the top-left when rotating an infinitely thin picture)
  • Removed check for effect mode when setting zoom or width/height.

One thing to note when comparing Maniac and EasyRPG: the former doesn't seem to properly clip the sprite in the corners when rotating the sprite.

From Maniac Patch version 240423:
[RPG_RT.exe]
Support for different magnification rates for special drawing of pictures (rotation, waveform, angle).

+ Added check if sprite's width/height is at zero. If so, skip the draw routine (helps avoid a glitchy sprite at the top-left when rotating an infinitely thin picture)
- Removed check for effect mode when setting zoom or width/height.

One thing to note when comparing Maniac and EasyRPG: the former doesn't seem to properly clip the sprite in the corners, when rotating the sprite.
@ToolMan2k ToolMan2k changed the title Maniac Patch : Remove scaling restrictions for pictures Maniacs Feature - Remove scaling restrictions for pictures Jul 24, 2024
@Ghabry
Copy link
Member

Ghabry commented Aug 1, 2024

I'm sorry for the lack of feedback on your PRs. We appreciate the contribution.

The project activity is usually reduced during summer because of vacation. I will review your code in September. Thank you for your patience.

@Ghabry Ghabry added this to the 0.8.1 milestone Sep 2, 2024
@Ghabry Ghabry merged commit 8c269b9 into EasyRPG:master Sep 2, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants