Skip to content

feat: sdk tween 'continuous' modes#5641

Merged
pravusjif merged 14 commits into
devfrom
feat/sdk-tween-continuous-modes
Oct 8, 2025
Merged

feat: sdk tween 'continuous' modes#5641
pravusjif merged 14 commits into
devfrom
feat/sdk-tween-continuous-modes

Conversation

@pravusjif

@pravusjif pravusjif commented Oct 1, 2025

Copy link
Copy Markdown
Member

WHY

3 new modes for the Tween component of the SDK have been designed and are defined at this shape up document.

We need that implementation on the 3 repos: Protocol, SDK, Explorer (this PR)

WHAT

  • Implemented support for the 3 new "continuous" modes of the PBTween component
  • Reimplemented all the Tween component test coverage that was a complete and utter disaster (world queries instead of get() calls, queries not even running due to entities not having the required archetype, wront tween mode used on wron tests...) + added test coverage for the new modes.

Related PRs:

TEST INSTRUCTIONS

  1. Download the build from this PR
  2. Enter the world nullrefxeption.dcl.eth
  3. Get close to the 3 elements in the center of the scene (cube, sphere, plane)
  4. Confirm that if you click the cube, it starts rotating and by clicking it again you stop it
  5. Confirm that if you click the sphere, it starts moving up and then it comes back down
  6. Confirm that if you click the plane the texture starts scrolling and you can toggle it by clicking it again
  7. Confirm that you can move to the top platform and the coin is rotating and when you touch it it disappears
Screen.Recording.2025-10-07.at.10.02.04.PM.mp4

@decentraland decentraland deleted a comment from github-actions Bot Oct 7, 2025
@pravusjif pravusjif added enhancement Enhancement of an existing feature do not merge sdk labels Oct 8, 2025
@github-actions

github-actions Bot commented Oct 8, 2025

Copy link
Copy Markdown
Contributor

@pravusjif pravusjif moved this from Todo to In Progress in Explorer Alpha Oct 8, 2025
@pravusjif
pravusjif marked this pull request as ready for review October 8, 2025 14:20
@pravusjif
pravusjif requested review from a team as code owners October 8, 2025 14:20
@pravusjif
pravusjif requested review from anicalbano and fcolarich and removed request for DafGreco, mikhail-dcl and popuz October 8, 2025 14:20
@pravusjif pravusjif moved this from In Progress to With QA / Awaiting Review in Explorer Alpha Oct 8, 2025

@anicalbano anicalbano left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

✔️ Reviewed by QA and working as expected both on Windows and Mac 😄

WINDOWS

08.10.2025_12.28.10_REC.mp4

MAC

08.10.2025_13.14.57_REC.mp4

Comment thread scripts/package.json Outdated

@fcolarich fcolarich left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good!

@github-project-automation github-project-automation Bot moved this from With QA / Awaiting Review to In Progress in Explorer Alpha Oct 8, 2025
@pravusjif pravusjif moved this from In Progress to With QA / Awaiting Review in Explorer Alpha Oct 8, 2025
@pravusjif
pravusjif enabled auto-merge (squash) October 8, 2025 22:28
@pravusjif
pravusjif merged commit 5e54720 into dev Oct 8, 2025
6 checks passed
@pravusjif
pravusjif deleted the feat/sdk-tween-continuous-modes branch October 8, 2025 23:04
@github-project-automation github-project-automation Bot moved this from With QA / Awaiting Review to Done in Explorer Alpha Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Enhancement of an existing feature sdk

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants