-
Notifications
You must be signed in to change notification settings - Fork 3
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 Surface viewer #4
Comments
Code looks amazing / everything re-factored. Will start having a look - and commenting - in a few hours. Bravo! |
Yeah it's quite different to the previous setup. The core package contains most of the ThreeJS / gbxml parsing code, whereas this is a react project. I'm happy to have a chat on slack or whatnot about react/webpack and co if it's not something you've come across before so you can have a proper play with it 😄 |
All good. And it might also be fun to see/meet each other. Are you on Google Hangouts or something? |
Aye indeed. I can do hangout or slack or what not. Shall we organise ourselves on slack or the forum? |
Good to close? |
Nope. Haven't started on this at all. |
Oh, I'm sorry I cluttered this issue. My bad, I thought https://www.ladybug.tools/spider-react/ was what you were referring to and therefore the issue had been dealt with. Now reading more carefully, I see you want to add more interaction. |
Add a small bit of UI that facilitates viewing details/data about highlights or clicked on surfaces.
The text was updated successfully, but these errors were encountered: