Skip to content

simplify sprite scrolling - #24

Merged
john-doherty merged 1 commit into
masterfrom
simplify-sprite-scrolling
Aug 5, 2026
Merged

simplify sprite scrolling#24
john-doherty merged 1 commit into
masterfrom
simplify-sprite-scrolling

Conversation

@john-doherty

@john-doherty john-doherty commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Sprite scrolling methods were confusing. This PR replaces parallaxScrollX, parallaxScrollY, and loopScrollX with a single scroll(speedX, speedY, options) method. Same runtime behaviour, fewer concepts.

Before:

cloud.parallaxScrollX(-0.5, 2000);
grass.loopScrollX(-2);

After:

cloud.scroll(-0.5, 0, { respawnAfter: 2000, offset: 100 });
grass.scroll(-2, 0, { loop: true });

Options:

  • loop — wrap by sprite size for seamless tiled backgrounds
  • respawnAfter — ms delay before reappearing on the opposite edge
  • offset — max random distance beyond the edge on respawn (0 = exact edge)

@john-doherty john-doherty self-assigned this Aug 5, 2026
@john-doherty
john-doherty merged commit 8985a3d into master Aug 5, 2026
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.

1 participant