Skip to content

Camera utility function#2273

Merged
wjt merged 2 commits into
mainfrom
camera-utility-function
Jun 2, 2026
Merged

Camera utility function#2273
wjt merged 2 commits into
mainfrom
camera-utility-function

Conversation

@manuq
Copy link
Copy Markdown
Collaborator

@manuq manuq commented Jun 2, 2026

grappling_hook_needles: Copy camera limits before the animation

Otherwise part of the scene outside the limits can be seen during the
transition. Also:

  • Remove an unwanted comment
  • Add 1 second delay before displaying the dialogue again
  • Use sine transition for the tween resource of the PhantomCamera2D

Add utility script for camera

The Phantom Camera addon doesn't have a way to keep the same limits as the
active camera before switching. So this has to be done though scripting.

Move the function in player_hook.gd into a utility script, and make it static.

manuq added 2 commits June 2, 2026 12:07
The Phantom Camera addon doesn't have a way to keep the same limits as the
active camera before switching. So this has to be done though scripting.

Move the function in player_hook.gd into a utility script, and make it static.
Otherwise part of the scene outside the limits can be seen during the
transition. Also:

- Remove an unwanted comment
- Add 1 second delay before displaying the dialogue again
- Use sine transition for the tween resource of the PhantomCamera2D
@manuq manuq marked this pull request as ready for review June 2, 2026 15:12
@manuq manuq requested a review from a team as a code owner June 2, 2026 15:12
@wjt wjt merged commit 6337812 into main Jun 2, 2026
6 checks passed
@wjt wjt deleted the camera-utility-function branch June 2, 2026 15:16
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 2, 2026

Play this branch at https://play.threadbare.game/branches/endlessm/camera-utility-function/.

(This launches the game from the start, not directly at the change(s) in this pull request.)

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.

2 participants