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

Add documentation to EditorExportPlatformWeb #86977

Conversation

Mickeon
Copy link
Contributor

@Mickeon Mickeon commented Jan 8, 2024

This PR fills in the class reference of EditorExportPlatformWeb. Yeah.

This also has the side-effect of displaying them when hovering over the settings in the Export window, which is really, really good:
image

Would be nice to get some feedback on factual correctness, sooner than later.


Sidenote, isn't it funny that html/experimental_virtual_keyboard has been "experimental" for 2 years? What a funny setting name.

@Mickeon Mickeon requested a review from a team as a code owner January 8, 2024 22:46
@Mickeon Mickeon force-pushed the documentation-editor-export-platform-web-no-funny-quirks-in-this-branch branch 3 times, most recently from 65f1070 to 4e38071 Compare January 8, 2024 23:09
@AThousandShips AThousandShips added this to the 4.3 milestone Jan 9, 2024
@AThousandShips AThousandShips added cherrypick:4.1 Considered for cherry-picking into a future 4.1.x release cherrypick:4.2 Considered for cherry-picking into a future 4.2.x release labels Jan 9, 2024
@Mickeon Mickeon force-pushed the documentation-editor-export-platform-web-no-funny-quirks-in-this-branch branch from 4e38071 to 7ffacb5 Compare February 11, 2024 17:28
@Mickeon
Copy link
Contributor Author

Mickeon commented Feb 11, 2024

Addressed all of the above feedback.

Copy link
Collaborator

@Faless Faless left a comment

Choose a reason for hiding this comment

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

LGTM!

@akien-mga akien-mga merged commit 0178cd4 into godotengine:master Feb 12, 2024
16 checks passed
@akien-mga
Copy link
Member

Thanks!

@Mickeon Mickeon deleted the documentation-editor-export-platform-web-no-funny-quirks-in-this-branch branch February 13, 2024 15:55
@akien-mga
Copy link
Member

Cherry-picked for 4.2.2.
Cherry-picked for 4.1.4.

@akien-mga akien-mga removed cherrypick:4.1 Considered for cherry-picking into a future 4.1.x release cherrypick:4.2 Considered for cherry-picking into a future 4.2.x release labels Mar 11, 2024
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.

5 participants