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

docs(v2:) Divide markdown-features/code-blocks to smaller sections #5061

Merged
merged 2 commits into from
Jun 25, 2021

Conversation

ramiy
Copy link
Contributor

@ramiy ramiy commented Jun 25, 2021

This PR adds few more heading to split the content to small topics.

This way users like me will be able to scroll directly to the specific section, instead of reading all the documentation.

I add to read the entire section because It was not divided into sections. Not a great user experience.

This PR adds few more heading to split the content to small topics.

This way users like me will be able to scroll directly to the specific section, instead of reading all the documentation.

I add to read the entire section because It was not divided into sections. Not a great user experience.
@ramiy ramiy requested review from lex111 and slorber as code owners June 25, 2021 09:19
@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Jun 25, 2021
@ramiy
Copy link
Contributor Author

ramiy commented Jun 25, 2021

Code-blocks-Docusaurus

@netlify
Copy link

netlify bot commented Jun 25, 2021

✔️ [V2]
Built without sensitive environment variables

🔨 Explore the source changes: d286b91

🔍 Inspect the deploy log: https://app.netlify.com/sites/docusaurus-2/deploys/60d5c35ef92a2d0008254641

😎 Browse the preview: https://deploy-preview-5061--docusaurus-2.netlify.app

@github-actions
Copy link

github-actions bot commented Jun 25, 2021

⚡️ Lighthouse report for the changes in this PR:

Category Score
🟠 Performance 66
🟢 Accessibility 98
🟢 Best practices 100
🟢 SEO 100
🟢 PWA 95

Lighthouse ran on https://deploy-preview-5061--docusaurus-2.netlify.app/

@@ -142,6 +146,8 @@ html[data-theme='dark'] .docusaurus-highlight-code-line {
}
```

## Multiple line highlighting {#multiple-line-highlighting}
Copy link
Collaborator

@Josh-Cena Josh-Cena Jun 25, 2021

Choose a reason for hiding this comment

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

I think the structure should be more like:

## Line highlighting
### Multiple line highlighting
### Highlighting with comments

Also, be ware that the section titles should be in sentence case.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch, thank you for noticing. It was fixed.

@slorber slorber added the pr: documentation This PR works on the website or other text documents in the repo. label Jun 25, 2021
@slorber slorber changed the title Divide markdown-features/code-blocks to smaller sections docs(v2:) Divide markdown-features/code-blocks to smaller sections Jun 25, 2021
@slorber
Copy link
Collaborator

slorber commented Jun 25, 2021

Thanks 👍

@slorber slorber merged commit 3fe7389 into facebook:master Jun 25, 2021
@ramiy ramiy deleted the patch-1 branch June 25, 2021 18:49
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: documentation This PR works on the website or other text documents in the repo.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants