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

Initial Docusaurus rewrite #1521

Merged
merged 20 commits into from
Jan 30, 2018
Merged

Initial Docusaurus rewrite #1521

merged 20 commits into from
Jan 30, 2018

Conversation

hzoo
Copy link
Member

@hzoo hzoo commented Jan 24, 2018

preview: https://deploy-preview-1521--babel.netlify.com/

Shouldn't look at the diff since there's too many files changed, looking for more PRs against it

cd website && yarn && yarn start

priority/easy


special pages

@babel-bot
Copy link
Contributor

babel-bot commented Jan 24, 2018

Deploy preview for babel ready!

Built with commit 587b885

https://deploy-preview-1521--babel.netlify.com

@xtuc
Copy link
Member

xtuc commented Jan 25, 2018

That is looking nice!

Should we keep our old design or change it by the way?

@hzoo
Copy link
Member Author

hzoo commented Jan 25, 2018

I don't think we should go out of our way to keep it so maybe both 😛 - vuejs.org, https://reactjs.org/, https://deploy-preview-3718--prettier.netlify.com/ look pretty nice @azz if you have ideas

I guess we just need to get the content (docs) back in working order and just continue to iterate otherwise it will take forever for this to happen and we need it asap for the content not to make it look nice.

@JoelMarcey
Copy link

@hzoo Hi! Thanks for giving Docusaurus a go. Let us know if you have any questions as you work through the conversion. You can jump into our Discord channel, file an issue on the Docusaurus repo, or just comment here.

cc: @ericnakagawa

@hzoo hzoo requested review from Daniel15, bvaughn and azz January 25, 2018 17:14
Copy link
Member

@Daniel15 Daniel15 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Anything particularly important to review in this diff? There's a lot of files :P

- name: npm
url: https://www.npmjs.com
logo: npm.svg
pinned: true
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this change related? If not, could you submit it as a separate PR?

@hzoo
Copy link
Member Author

hzoo commented Jan 25, 2018

Sorry it I didn't mean to ask to review the code perse since at the top I said it would be impossible to do in a PR. It was more about feedback on the UI and the other things - or if there is something not covered in my bullet points that you are aware of.

Or maybe related to deployment/versioning/redirects

@Daniel15
Copy link
Member

Daniel15 commented Jan 25, 2018

oh, lol, I'm not sure how I missed that.

This page doesn't work properly (has some markup showing): https://deploy-preview-1521--babel.netlify.com/docs/en/setup.html

One thing we should ensure is kept is URLs. For example, https://deploy-preview-1521--babel.netlify.com/docs/usage/caveats/ throws a 404 because the URL has changed to https://deploy-preview-1521--babel.netlify.com/docs/en/caveats.html.

If it's impossible to keep exactly the same URLs, then the old URLs should 301 (permanent) redirect to new ones. You can do this with Netlify by using a _redirects file: https://www.netlify.com/docs/redirects/. For the language-based pages in particular, Netlify supports language- and location-based redirects. For example, /docs/usage/caveats could redirect to /docs/en/caveats.html if the browser's language is set to English, /docs/it/caveats.html if it's set to Italian, etc. Each redirect needs to be specified in the _redirects file (one line per page per supported language).

@hzoo
Copy link
Member Author

hzoo commented Jan 25, 2018

@Daniel15 yeah i'm tracking the pages that don't work in the ^ but I didn't do a good job of explaining haha (setup is one of them)

yeah exactly we want to keep old urls and redirect 👍 👍

@hzoo
Copy link
Member Author

hzoo commented Jan 25, 2018

Ok made a new site on netlify so we can make a new branch to target for the old site: old-site and a new subdomain: https://old.babeljs.io for previous stuff

This was referenced Jan 26, 2018
@hzoo hzoo added the new site any issues for the new site using docusaurus label Jan 26, 2018
@hzoo hzoo mentioned this pull request Jan 26, 2018
@GregBorrelly
Copy link
Contributor

Hey guys, are there any tasks I can take over? I would love to participate.

@SavePointSam
Copy link
Contributor

Same here! I'd love to tackle some thing/s this weekend :D

@hzoo hzoo mentioned this pull request Jan 26, 2018
* feat(videos): convert the videos page into react component

* update: change video.css

* update: rm old videos file

* fix: add @media screen into video.css
@hzoo
Copy link
Member Author

hzoo commented Jan 26, 2018

Yeah @GregBorrelly, @SavePointSam, and anyone else looking - I already made a few issues tagged in this PR that people can do as separate tasks. It's mostly making sure that we can bring back functionality from the current site (babeljs.io) into this one (check the preview url), so anything you can do for that would be appreciated since it all adds up. Sometimes it's worth rewriting too if you feel inspired too.

@SavePointSam SavePointSam mentioned this pull request Jan 28, 2018
5 tasks
ajaymathur and others added 8 commits January 28, 2018 10:13
* added styles for user

* added browser hack for transform
* Add repl to new site

* Tweak styles

* Remove old things

* height: 100% fallback
* team page added.(#1523)

* custom page team added to header links

* unique key prop warning fixed.unwanted variable removed

* code refactoring, composed jsx elements into functional comps

* responsive design bug fixed

* container left,right padding removed for more usage of space

* common styles from video.css & team.css -> babel.css
* removed maincontainer from css style where it could be removed for users page

* updated styles?
@hzoo hzoo changed the title Docusaurus (WIP) Initial Docusaurus rewrite Jan 30, 2018
@hzoo hzoo merged commit 2f4e200 into master Jan 30, 2018
@hzoo
Copy link
Member Author

hzoo commented Jan 30, 2018

Will make a new issue #1485 (comment)

This was referenced Jan 30, 2018
@hzoo hzoo deleted the docusaurus branch February 3, 2018 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new site any issues for the new site using docusaurus
Projects
None yet
Development

Successfully merging this pull request may close these issues.