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
Cleanup: console logging, debug and testing output, lint. #486
Cleanup: console logging, debug and testing output, lint. #486
Conversation
✅ Deploy Preview for bldrs-share ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
src/Containers/CadView.jsx
Outdated
depthTest: true, | ||
}) | ||
const selectMat = new MeshLambertMaterial({ | ||
transparent: true, | ||
color: theme.palette.highlight.main, | ||
color: theme.palette.secondary.main, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So the highlighting that is active in production was broken but in a different way. The console log shows the highlight color is undefined.. it was being passed in from our theme, but the theme property wasn't defined. So, I picked what looked good to me last night :) but yeah, open to any suggestions for alternatives
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ptal |
There are conflicts. |
Fixes #448 |
No description provided.