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

Revert "fix(v2): remove auto wrap for code blocks (#2048)" #2071

Merged
merged 1 commit into from
Nov 30, 2019

Conversation

endiliey
Copy link
Contributor

@endiliey endiliey commented Nov 30, 2019

Motivation

As mentioned in #2056
and #2048 (comment)

I think it might be best to revert #2048 first. If we want to attempt on removing the codeblock autowrap, we can do it in new PR. Lot of changes has been added since alpha.36 and I think it'd be best to release soon so we can dogfood versioning & lot of other critical bugfix can land.

Because I'm seeing some unpleasant layout that is kinda zoomed and cut off. Notice the scrollbar
https://v2.docusaurus.io/docs/next/lifecycle-apis/
before- (#2048)

https://v2.docusaurus.io/docs/next/markdown-features
image

Have you read the Contributing Guidelines on pull requests?

yes

Test Plan

No longer zoomed
image

image

@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Nov 30, 2019
@docusaurus-bot
Copy link
Contributor

Deploy preview for docusaurus-2 ready!

Built with commit a4d9295

https://deploy-preview-2071--docusaurus-2.netlify.com

@docusaurus-bot
Copy link
Contributor

Deploy preview for docusaurus-preview ready!

Built with commit a4d9295

https://deploy-preview-2071--docusaurus-preview.netlify.com

Copy link
Contributor

@lex111 lex111 left a comment

Choose a reason for hiding this comment

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

I don't mind, but let's wait for a reaction from @yangshun

@endiliey
Copy link
Contributor Author

endiliey commented Nov 30, 2019

We can close this if we find a solution. This is quite a temporary solution, because I was hoping to release alpha.37 soon since there has been too many commits already 😆

I do have another concern though. CodeBlock is one of the most swizzled component in Docusaurus 2. Because to add new language, you need to swizzle CodeBlock. Need to remind user to update their CodeBlock css and code when I release new alpha version if we're not reverting.

@yangshun yangshun merged commit 39e9e75 into master Nov 30, 2019
@yangshun yangshun deleted the endi/revert2048 branch November 30, 2019 20:58
@yangshun
Copy link
Contributor

Let's revert this first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Signed Facebook CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants