-
-
Notifications
You must be signed in to change notification settings - Fork 390
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
PR for #297 #311
PR for #297 #311
Conversation
…issing from three.js - #297
} from 'three'; | ||
|
||
/** | ||
* this is missing in @types from three.js :( |
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.
May we add such things to separate file?
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.
best would be to fix @types/three
imo.
Unless there is a reason they don't provide interfaces for certain buffer geometries...
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.
A few code style fixes
export * from './Group'; | ||
export {Dodecahedron} from './Dodecahedron'; | ||
export * from './Extrude'; | ||
export {Icosahedron} from './Icosahedron'; |
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.
What is special with Dodecahedron
and Icosahederon
so they are exported different way ?
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.
I think it's because I didn't want to export other class/interfaces from Dode/Ico.
}, | ||
|
||
buffer: true | ||
}); |
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.
\n
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.
you fixed those right?
types/whs-tests.ts
Outdated
thetaSegments: 10, | ||
thetaStart: 0 | ||
} | ||
}); |
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.
\n
}, | ||
|
||
buffer: true | ||
}); |
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.
\n
}, | ||
|
||
buffer: true | ||
}); |
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.
\n
}, | ||
|
||
buffer: true | ||
}); |
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.
\n
}, | ||
|
||
buffer: true | ||
}); |
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.
\n
}, | ||
|
||
buffer: true | ||
}); |
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.
\n
geometry: { | ||
points: [new Vector2(1, 1)] | ||
} | ||
}); |
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.
\n
radius: 2, | ||
detail: 0.2 | ||
} | ||
}); |
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.
\n
- Removed deprecated useCustomMaterial - Fixed line padding
@hirako2000 Fixed code style & updated branch. |
Thanks, I think we are fine to merge this. we can clean up better as we go along. I intend to do some proper testing with the typescript boilerplate. Im sure I will find some issues with some types. |
It's a lot.
I suggest to just glance over it and improve later where appropriate.