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

feat(v2): bootstrap theme, preset, template, CI previews #2981

Merged
merged 17 commits into from
Aug 17, 2020

Conversation

fanny
Copy link
Contributor

@fanny fanny commented Jun 22, 2020

Motivation

Document the bootstrap template, preset and theme

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Bonus points for screenshots and videos!)

Related PRs

(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

@fanny fanny requested a review from yangshun as a code owner June 22, 2020 22:54
@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Jun 22, 2020
@docusaurus-bot
Copy link
Contributor

docusaurus-bot commented Jun 22, 2020

Deploy preview for docusaurus-2 ready!

Built with commit 1dcf606

https://deploy-preview-2981--docusaurus-2.netlify.app

@slorber
Copy link
Collaborator

slorber commented Jul 29, 2020

🤪 just understood that is theme is for Bootstrap the CSS framework.

I thought it was about providing the basic theme structure to be able to render docs, blogs etc ^^

Is this PR ready for review or WIP?

@fanny
Copy link
Contributor Author

fanny commented Aug 8, 2020

🤪 just understood that is theme is for Bootstrap the CSS framework.

I thought it was about providing the basic theme structure to be able to render docs, blogs etc ^^

Is this PR ready for review or WIP?

Sorry for the delay, you're right. And Is Ready for review

@slorber
Copy link
Collaborator

slorber commented Aug 11, 2020

@fanny what's your workflow to test the bootstrap theme btw? I've never seen it in action

@yangshun
Copy link
Contributor

It should be just going into the Bootstrap template, run the site and visually inspect

@slorber
Copy link
Collaborator

slorber commented Aug 12, 2020

I see thanks.

One idea I had was to make our deploy previews deploy D2 website in multiple variants, like theme classic, theme bootstrap etc... that would dogfood and ensure this theme keeps the feature parity with the classic theme and stays in sync over time. Will see how this can be made soon

@yangshun
Copy link
Contributor

That would be fantastic :)

@slorber
Copy link
Collaborator

slorber commented Aug 17, 2020

@yangshun @fanny it's not perfect but got it running :)

https://deploy-preview-2981--docusaurus-2.netlify.app/classic/
https://deploy-preview-2981--docusaurus-2.netlify.app/bootstrap/

Both will be deployed on each PR now! Will make it easier to iterate on the bootstrap theme.

Note: I disabled versioning for bootstrap previews, so that it does not increase too much the build time ;)

@slorber slorber changed the title doc(v2): bootstrap theme, preset, template feat(v2): bootstrap theme, preset, template Aug 17, 2020
@slorber slorber changed the title feat(v2): bootstrap theme, preset, template feat(v2): bootstrap theme, preset, template, CI previews Aug 17, 2020
@slorber slorber added the pr: new feature This PR adds a new API or behavior. label Aug 17, 2020
@slorber slorber merged commit 8c05aa8 into master Aug 17, 2020
@yangshun yangshun deleted the fanny/bootstrap-doc branch August 26, 2020 01:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Signed Facebook CLA pr: new feature This PR adds a new API or behavior.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants