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

Fixed typos in the Documentation #28

Open
wants to merge 1 commit into
base: public/0.11.x
Choose a base branch
from

Conversation

SilasContaifer
Copy link

I saw that inside the scripts like in UxtHandConstraintComponent.h we refer to the component as **UU**xtHandConstraintComponent but when a user uses Unreal do insert this component, they cannot search for anything that starts with the UUx... as for them each component will be find it with their name starting with Uxt (e.g. Uxt Hand Constraint).

So I propose this changes removing the double U in this three files in the documentation, I confirm that that is no other citation of a component starting with UUxt in the other .md in this Docs folder.

I saw that inside the scripts like in UxtHandConstraintComponent.h we refer to the component as UUxtHandConstraintComponent but when a user uses Unreal do insert this component, they cannot search for anything that starts with the UUx... as for them each component will be find it with their name starting with Uxt (e.g. Uxt Hand Constraint)
@ghost
Copy link

ghost commented Mar 2, 2021

CLA assistant check
All CLA requirements met.

@luis-valverde-ms
Copy link

Thanks a lot @SilasContaifer . I'm running it through the team to make sure we standardize the way we write type names in docs. My vote goes for what you propose (skip type prefix, at least in actors and components). Once we reach a decision, I'll update you here, implement the changes in the internal repo and push them to GitHub.

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

Successfully merging this pull request may close these issues.

None yet

2 participants