Skip to content

Initial Docusaurus rewrite - #1521

Merged
hzoo merged 20 commits into
masterfrom
docusaurus
Jan 30, 2018
Merged

Initial Docusaurus rewrite#1521
hzoo merged 20 commits into
masterfrom
docusaurus

Conversation

@hzoo

@hzoo hzoo commented Jan 24, 2018

Copy link
Copy Markdown
Member

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

babel-bot commented Jan 24, 2018

Copy link
Copy Markdown
Contributor

Deploy preview for babel ready!

Built with commit 587b885

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

@xtuc

xtuc commented Jan 25, 2018

Copy link
Copy Markdown
Member

That is looking nice!

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

@hzoo

hzoo commented Jan 25, 2018

Copy link
Copy Markdown
Member Author

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
Copy Markdown

@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, azz and bvaughn January 25, 2018 17:14

@Daniel15 Daniel15 left a comment

Copy link
Copy Markdown
Member

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

Comment thread _data/users.yml
- name: npm
url: https://www.npmjs.com
logo: npm.svg
pinned: true

Copy link
Copy Markdown
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

hzoo commented Jan 25, 2018

Copy link
Copy Markdown
Member Author

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

Daniel15 commented Jan 25, 2018

Copy link
Copy Markdown
Member

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

hzoo commented Jan 25, 2018

Copy link
Copy Markdown
Member Author

@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

hzoo commented Jan 25, 2018

Copy link
Copy Markdown
Member Author

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
Copy Markdown
Contributor

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

@SavePointSam

Copy link
Copy Markdown
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

hzoo commented Jan 26, 2018

Copy link
Copy Markdown
Member Author

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

hzoo commented Jan 30, 2018

Copy link
Copy Markdown
Member Author

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.