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

Prepared Autoload(Singletons) demo for Godot 4 #702

Merged
merged 1 commit into from
Apr 3, 2022
Merged

Prepared Autoload(Singletons) demo for Godot 4 #702

merged 1 commit into from
Apr 3, 2022

Conversation

voylin
Copy link
Contributor

@voylin voylin commented Mar 31, 2022

Not really any fixing were required, some data changed between versions
when opening and testing everything. Did some minor changes to the project
settings so the new Godot 4 terms are used.

Everything works fine without problems.

@@ -4,7 +4,7 @@ This demo shows how to use autoloads to change between scenes.

Language: GDScript

Renderer: GLES 2
Renderer: Vulkan
Copy link
Member

Choose a reason for hiding this comment

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

If we're updating this text, we should be specifying if it's Vulkan Clustered or Vulkan Mobile.

Additionally, we should be setting demos to Vulkan Mobile if it doesn't look any different compared to Vulkan Clustered. Especially for GUI-only demos, they will look the same in Vulkan Mobile. Go to Project -> Project Settings -> General tab -> Rendering section -> Vulkan -> Back End -> Select Forward Mobile.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks! Fixed, squashed and pushed!

@aaronfranke aaronfranke added this to the 4.0 milestone Apr 3, 2022
Not really any fixing were required, some data changed between versions
when opening and testing everything. Did some changes to the project
settings (2d to canvas_items). Everything works fine without problems.

Change to Vulkan Mobile
Copy link
Member

@aaronfranke aaronfranke left a comment

Choose a reason for hiding this comment

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

Thanks!

@aaronfranke aaronfranke merged commit 3eb81eb into godotengine:4.0-dev Apr 3, 2022
@voylin voylin deleted the Fix_Autoload_Demo_4.0-dev branch April 3, 2022 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants