Skip to content

Clarify align() behavior regarding force_update_scroll()#114638

Merged
akien-mga merged 1 commit into
godotengine:masterfrom
BenEklundCS:godot-docs-7307-camera2d-align-note
Jan 9, 2026
Merged

Clarify align() behavior regarding force_update_scroll()#114638
akien-mga merged 1 commit into
godotengine:masterfrom
BenEklundCS:godot-docs-7307-camera2d-align-note

Conversation

@BenEklundCS
Copy link
Copy Markdown
Contributor

As noted in this docs issue, the Camera2D source code shows that align() calls _update_scroll(), which is also called by force_update_scroll(). This adds a note to the documentation to clarify that calling force_update_scroll() separately is not required.

Copy link
Copy Markdown
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

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

Looks good to me.

The CI failure is unrelated (Maven issue).

@akien-mga akien-mga changed the title Clarify align() behavior regarding force_update_scroll() Clarify align() behavior regarding force_update_scroll() Jan 8, 2026
@akien-mga akien-mga merged commit f1fad38 into godotengine:master Jan 9, 2026
39 of 40 checks passed
@akien-mga
Copy link
Copy Markdown
Member

Thanks! And congrats for your first merged Godot contribution 🎉

@BenEklundCS
Copy link
Copy Markdown
Contributor Author

Thank you for the review and merge! Excited to contribute more.

rivie13 pushed a commit to rivie13/Phoenix-Agentic-Engine that referenced this pull request Feb 16, 2026
…7-camera2d-align-note

Clarify `align()` behavior regarding `force_update_scroll()`
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.

4 participants