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

Scale parallax layers perspectively-correctly #2954

Merged
merged 6 commits into from
Jul 27, 2024

Conversation

Hume2
Copy link
Member

@Hume2 Hume2 commented May 25, 2024

I implemented it for both tilemaps and backgrounds. DEMO: https://www.youtube.com/watch?v=GnFArTYBL4E

Scaling now works so that it imitates a perspective camera. The layers scale differently based on their scrolling speed.

Also note that the z-position based on which the scaling is calculated, does not depend on the "z-pos" value at all, so it is nicely compatible with the old levels.

It also fixes the bug which allows players to see around paralax layers when the viewport is scaled down. If the layer is large enough to cover the level in the default scale, it now works in all scales.

@tobbi tobbi changed the title Scale paralax layers perspectively-correctly Scale parallax layers perspectively-correctly Jun 8, 2024
@mrkubax10 mrkubax10 added type:patch category:code status:needs-review Work needs to be reviewed by other people labels Jun 15, 2024
src/object/tilemap.cpp Outdated Show resolved Hide resolved
src/object/tilemap.cpp Outdated Show resolved Hide resolved
src/video/drawing_context.cpp Outdated Show resolved Hide resolved
src/video/drawing_context.cpp Outdated Show resolved Hide resolved
src/object/tilemap.cpp Outdated Show resolved Hide resolved
Copy link
Member

@MatusGuy MatusGuy left a comment

Choose a reason for hiding this comment

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

Untested

@Rusty-Box
Copy link
Member

Could you sync your branch so I can make use of the portable zip for windows builds please? Makes it more convenient for me to test

Copy link
Member

@Rusty-Box Rusty-Box left a comment

Choose a reason for hiding this comment

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

Love this

src/object/background.cpp Outdated Show resolved Hide resolved
@MatusGuy MatusGuy merged commit 5a50578 into SuperTux:master Jul 27, 2024
33 checks passed
@mrkubax10 mrkubax10 removed the status:needs-review Work needs to be reviewed by other people label Jul 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants