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(en): clarify code highlighting & fix typos #1604

Closed
wants to merge 1 commit into from
Closed

docs(en): clarify code highlighting & fix typos #1604

wants to merge 1 commit into from

Conversation

postmodernistx
Copy link
Contributor

Removes extra spaces in code samples.

Relates to #1534.

Removed extra spaces in code samples; highlighting didn't work with them included.

Relates to #1534.
@brc-dd brc-dd added the invalid This doesn't seem right label Nov 13, 2022
@@ -392,7 +392,7 @@ Additionally, you can define a number of lines to focus using `// [!code focus:<
export default {
data () {
return {
msg: 'Focused!' // [!codeㅤ focus]
msg: 'Focused!' // [!codeㅤfocus]
Copy link
Member

@brc-dd brc-dd Nov 13, 2022

Choose a reason for hiding this comment

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

This is not a space either. It's U+3164. Having such characters in source code is worse than having two spaces. It is more likely to go unnoticed than two spaces!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh right, didn't see that - nor did my editor apparently! Thanks for pointing it out.

Copy link
Member

@brc-dd brc-dd Nov 16, 2022

Choose a reason for hiding this comment

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

Yeah, I missed that too. It is there in the current docs too. I'd missed them while reviewing earlier PRs. Gonna remove all those in #1615 itself. Removing two spaces won't work as it will make shiki highlight those blocks too. We can however add a note there that users should remove an extra space if copying.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think mentioning removing the extra spaces is helpful because it took me about 20 minutes to figure out why on earth it didn't work 😅

@brc-dd brc-dd closed this Nov 13, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants