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

feat(templates/threejs): easier to use template #2473

Merged
merged 7 commits into from
Mar 22, 2024

Conversation

qbzzt
Copy link
Contributor

@qbzzt qbzzt commented Mar 19, 2024

Closing: #2469

Copy link

changeset-bot bot commented Mar 19, 2024

⚠️ No Changeset found

Latest commit: 211599a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

<meshStandardMaterial color="green" />
</mesh>
<mesh position={[0, -1, 0]}>
{/* eslint-disable-next-line react/no-unknown-property */}
Copy link
Member

Choose a reason for hiding this comment

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

can we figure out how to get react+eslint to allow this? I assume there's a three.js eslint config to include here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

And the solution I found online doesn't work. Can we please just not worry about it? Prettier here is more trouble than it is worth, IMAO.

Copy link
Contributor

Choose a reason for hiding this comment

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

I've always run into this issue with ThreeJS and TypeScript, I'll play around and try fix it. Also I get errors for all the elements so we are going to need a lot of these comments otherwise!
image

Copy link
Contributor

@yonadaaa yonadaaa Mar 20, 2024

Choose a reason for hiding this comment

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

Sounds like an open issue and the @react-three/eslint-plugin didn't work for me.

For now I added a comment at the top of the file but we could also add a eslintrc rule like

  ts"react/no-unknown-property": ["off", { "ignore": ["JSX"] }],

yonadaaa
yonadaaa previously approved these changes Mar 20, 2024
@yonadaaa
Copy link
Contributor

Tweaked the topbar colors:
image

@yonadaaa yonadaaa merged commit e08b616 into main Mar 22, 2024
11 checks passed
@yonadaaa yonadaaa deleted the 240319-update-3js-template branch March 22, 2024 11:28
@holic
Copy link
Member

holic commented Mar 22, 2024

@yonadaaa need a changeset for "create-mud" to make sure this goes out with next release

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.

3 participants