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

Adds ability to run frontend without the table or controller #2

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

jontonsoup4
Copy link

Description

Loved your video! I wanted to get the frontend code running without having to have the table or a controller plugged in, and what started as a small test ended up in this PR. Every change I made was in the frontend, and this will allow more people to see and interact with your cool project.

Here's a deployed version of the app!

Changes

  • Added a README
    • Added a deployed demo link to the readme
    • Added installation instructions
  • Fixed animation loop to allow for running without a connection to the server or a controller
  • Fixed rendering bug to allow for higher resolution rendering
  • Leverage Vite for building instead of the http-server
  • Updates Vite config (window will open automatically when server starts and the asset resolution is fixed)
  • Fixes the outdated three/examples imports
  • Adds globalControls const to use for lil-gui debugging
  • Adds dampening and maxPolarAngle to OrbitControls and removes panning
  • Removed unused code and console.logs
  • Ran prettier over everything, so misc code styling changes

New Packages

  • Adds lil-gui for debugging/fun
  • Added prettier for code formatting. I kept most of the stylistic choices you already had

Removed Packages

  • Removes http-server in favor of vite

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.

1 participant