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

Extend Roact JSX documentation #100

Merged
merged 1 commit into from
Dec 29, 2021
Merged

Extend Roact JSX documentation #100

merged 1 commit into from
Dec 29, 2021

Conversation

Asmeili
Copy link
Contributor

@Asmeili Asmeili commented Dec 20, 2021

I personally struggled with finding out this information, and think it would be quite useful to include it within the documentation.

It might need some rewording, however this is mainly to illustrate what I'm trying to add into the documentation and what use-cases it might have.
The example seems simple yet is not being explained, a developer that isn't aware of IntrinsicElements or it's purpose might be confused here - should I extend the section to quickly explain why IntrinsicElements is being extended and how it relates to JSX (Or link to an external resource that does this)?

For JSX naming conventions, perhaps instead of explaining it link to https://roblox-ts.com/docs/guides/roact-jsx/#tag-names?

More feedback needed on this. 😊

I personally struggled with finding out this information, and think it would be quite useful to include it within the documentation.

It might need some rewording, however this is mainly to illustrate what I'm trying to add into the documentation and what use-cases it might have.
The example seems simple yet is not being explained, a developer that isn't aware of ``IntrinsicElements`` or it's purpose might be confused here - should I extend the section to quickly explain why ``IntrinsicElements`` is being extended and how it relates to ``JSX`` (Or link to an external resource that does this)?

For JSX naming conventions, perhaps instead of explaining it link to https://roblox-ts.com/docs/guides/roact-jsx/#tag-names?


More feedback needed on this. 😊
@osyrisrblx osyrisrblx merged commit 2eedc4f into roblox-ts:master Dec 29, 2021
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.

2 participants