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.

Sign up for GitHub

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

Jump to bottom

Document unsupported features in Mobile and Compatibility rendering methods #72659

Merged
merged 1 commit into from
Feb 13, 2023
Merged

Document unsupported features in Mobile and Compatibility rendering methods #72659

merged 1 commit into from
Feb 13, 2023
+72 −41

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Feb 3, 2023 •
edited
Loading

This also contains miscellaneous clarifications and improvements, especially related to performance in material settings.

This partially addresses #55880 (it does not add node configuration warnings where appropriate).

@Calinou Calinou requested a review from a team as a code owner February 3, 2023 11:51
@Calinou Calinou added this to the 4.0 milestone Feb 3, 2023
@Calinou Calinou force-pushed the doc-rendering-method-support branch 2 times, most recently from 51a4055 to 1cdc936 Compare February 3, 2023 11:56
@akien-mga akien-mga requested a review from a team February 3, 2023 14:42
clayjohn
clayjohn reviewed Feb 3, 2023
View reviewed changes
doc/classes/Environment.xml Outdated Show resolved Hide resolved
doc/classes/NinePatchRect.xml Outdated Show resolved Hide resolved
doc/classes/ProjectSettings.xml Outdated Show resolved Hide resolved
@Calinou Calinou force-pushed the doc-rendering-method-support branch 2 times, most recently from b354024 to 60ca635 Compare February 3, 2023 22:56
@Calinou Calinou mentioned this pull request Feb 8, 2023
Closed
@Calinou Calinou force-pushed the doc-rendering-method-support branch from 60ca635 to 677c4d5 Compare February 8, 2023 14:39
@godotengine godotengine deleted a comment from Ahmad-Raza18 Feb 8, 2023
@clayjohn
Copy link
Member

This should be good to merge once the suggested change from CaelusV is made

@Calinou Calinou requested a review from a team as a code owner February 13, 2023 21:16
@Calinou
Copy link
Member Author

Calinou commented Feb 13, 2023

This should be good to merge once the suggested change from CaelusV is made

Done. I've also fixed two other occurrences of [b]Note[/b] where the : was outside the [b] tag (unlike all other occurrences in the class reference).

@akien-mga akien-mga merged commit 4848877 into godotengine:master Feb 13, 2023
@akien-mga
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CaelusV CaelusV CaelusV left review comments

@clayjohn clayjohn clayjohn approved these changes

Assignees
No one assigned
Labels
documentation enhancement topic:rendering
Projects
None yet
Milestone
4.0
Development

Successfully merging this pull request may close these issues.
5 participants
@Calinou @clayjohn @akien-mga @CaelusV @YuriSizov