-
Notifications
You must be signed in to change notification settings - Fork 701
[Docs] New website #1109
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
[Docs] New website #1109
Conversation
|
Checked out https://formidable-com-spectacle-staging-1109.surge.sh/open-source/spectacle/ a bit and looks great so far!!! |
removing the requirement for noMargin and noPadding misc code cleanup
cleanup and consistency
i have removed the autoPlay prop as it crashes the app at the terminal. made some comments as a reminder of the issue
gksander
left a comment
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.
This is a chonky PR, but this thing is slick. I dropped a few comments here for us to address real quick, but nothing major. I think we do a little bit of tidying up (I can actually get started on that), get a final design stamp of approval on this, and ship this thing!
gksander
left a comment
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.
LGTM, LFG 🚀
Description
react-staticsite with a new one backed bydocusaurus./docsand all docusaurus-related code is located in the new/websitefolder.Type of Change
How Has This Been Tested?
Local testing through
yarn startin/website.Checklist: (Feel free to delete this section upon completion)