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

Make the rendering method dropdown also affect mobile if compatible #72461

Merged

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Jan 31, 2023

Follow-up to #72460.

This prevents visual discrepancies between desktop and mobile platforms after switching rendering methods using this dropdown.

The restart dialog now displays which rendering methods will be used, and the dropdown tooltip also lists caveats related to specific platforms.

I've tested all switching combinations and can confirm it works as expected. Note that project.godot will not store default values, so you won't see the mobile rendering method being visibly stored in that file for the .mobile override (but gl_compatibility will).

Preview

Screenshot_20240103_183552

Screenshot_20240103_183824

@Calinou
Copy link
Member Author

Calinou commented Sep 20, 2023

More discussion on the subject: godotengine/godot-proposals#7767

This prevents visual discrepancies between desktop and mobile platforms
after switching rendering methods using this dropdown.

The restart dialog now displays which rendering methods will be used,
and the dropdown tooltip also lists caveats related to specific platforms.
@Calinou Calinou force-pushed the editor-renderer-select-mobile branch from 62bbfd8 to 57f3bdb Compare January 3, 2024 17:39
Copy link
Member

@clayjohn clayjohn left a comment

Choose a reason for hiding this comment

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

Looks great! Tested locally and it works well. This should really help avoid confusion

@clayjohn clayjohn modified the milestones: 4.x, 4.3 Jan 6, 2024
@clayjohn clayjohn added the cherrypick:4.2 Considered for cherry-picking into a future 4.2.x release label Jan 6, 2024
@akien-mga akien-mga merged commit 15186b6 into godotengine:master Jan 8, 2024
15 checks passed
@akien-mga
Copy link
Member

Thanks!

@Calinou Calinou deleted the editor-renderer-select-mobile branch January 8, 2024 19:47
@YuriSizov YuriSizov removed the cherrypick:4.2 Considered for cherry-picking into a future 4.2.x release label Jan 24, 2024
@YuriSizov
Copy link
Contributor

Cherry-picked for 4.2.2.

@YuriSizov YuriSizov removed the cherrypick:4.1 Considered for cherry-picking into a future 4.1.x release label Jan 24, 2024
@YuriSizov
Copy link
Contributor

Cherry-picked for 4.1.4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants