Skip to content

Conversation

DePasqualeOrg
Copy link
Contributor

No description provided.

@@ -129,7 +129,7 @@ const imgUrl = new URL('./img.png', import.meta.url).href
document.getElementById('hero-img').src = imgUrl
```

This works natively in modern browsers - in fact, Vite doesn't need to process this code at all during development!
This works natively in modern browsers in fact, Vite doesn't need to process this code at all during development!
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
This works natively in modern browsers in fact, Vite doesn't need to process this code at all during development!
This works natively in modern browsers - in fact, Vite doesn't need to process this code at all during development!

I'm reverting this for now as we have many places with the same pattern. (And personally I don't think it's a big issue to fix in practice).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hyphens are distinct from m dashes and n dashes and should not be used as a substitute for these long dashes.

@bluwy bluwy changed the title docs: Minor fixes for punctuation and grammar docs: minor fixes for punctuation and grammar Aug 20, 2025
@bluwy bluwy merged commit aa3323d into vitejs:main Aug 20, 2025
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants