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

Update Camera 2D to fix and remove some notifications #43993

Closed
wants to merge 7 commits into from
Closed

Update Camera 2D to fix and remove some notifications #43993

wants to merge 7 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Nov 30, 2020

This simple pull request will fix : #28492
By removing the if statement that stops the camera from updating using NOTIFICATION_TRANSFORM_CHANGED .
And removing the NOTIFICATION_INTERNAL_PROCESS as it has no use.
As well as NOTIFICATION_PROCESS

@Chaosus Chaosus added this to the 4.0 milestone Nov 30, 2020
@Chaosus Chaosus changed the title Update Camera 2D to fix and remove some notifications (Master) Update Camera 2D to fix and remove some notifications Nov 30, 2020
@Chaosus
Copy link
Member

Chaosus commented Nov 30, 2020

Thanks for your first-time contribution - In order to accept your PR - you need to merge all commits together as described here: https://docs.godotengine.org/en/latest/community/contributing/pr_workflow.html#the-interactive-rebase

@ghost ghost requested a review from vnen as a code owner November 30, 2020 19:43
@ghost
Copy link
Author

ghost commented Nov 30, 2020

Looks like I have Destroyed this PR.
I know not how to fix it, there for am going to close it and reopen it.

@ghost ghost closed this Nov 30, 2020
This pull request was closed.
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.

Camera2d has a one frame delay
1 participant