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.

Sign up for GitHub

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

Jump to bottom

Svelte 5: invalid CSS identifier caused by a line break #9795

Closed
mquandalle opened this issue Dec 5, 2023 · 0 comments · Fixed by #9806
Closed

Svelte 5: invalid CSS identifier caused by a line break #9795

mquandalle opened this issue Dec 5, 2023 · 0 comments · Fixed by #9806

Comments

@mquandalle
Copy link

Describe the bug

The following code throws an exception:

<style>
	:global(
		body
	) {
		background: yellow;
		height: 100vh;
	}
</style>

Expected a valid CSS identifier

Reproduction

https://svelte-5-preview.vercel.app/#H4sIAAAAAAAAEy2NwQqDMBBEf0X21EKh9ppKod_ReFCzJqFrImZjCZJ_r6m9zXsDMxuMljCAeG3guglBwHOe4QKc5gJhRWLcOfi4DMU0gRPhQzrJQpPvOzqVLLn3KpV0rra_6Ia3Xnx0SlQJifznfhQGrTYsqltdr-bnsnTN9Rjevyav7GhRgeAlYm7zF5v574qmAAAA

Logs

No response

System Info

Svelte 5

Severity

annoyance
@mquandalle mquandalle changed the title Svelte 5: invalid CSS selector caused by an authorized line break Svelte 5: invalid CSS selector caused a line break Dec 5, 2023
@mquandalle mquandalle changed the title Svelte 5: invalid CSS selector caused a line break Svelte 5: invalid CSS selector caused by a line break Dec 5, 2023
@mquandalle mquandalle changed the title Svelte 5: invalid CSS selector caused by a line break Svelte 5: invalid CSS identifier caused by a line break Dec 5, 2023
dummdidumm added a commit that referenced this issue Dec 6, 2023
@dummdidumm
fix: tweak css nth regex
fc8bec4 
fixes #9795
@dummdidumm dummdidumm mentioned this issue Dec 6, 2023
Merged
5 tasks
dummdidumm added a commit that referenced this issue Dec 6, 2023
@dummdidumm
fix: tweak css nth regex (#9806)
3c2e656 
fixes #9795
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: tweak css nth regex sveltejs/svelte
1 participant
@mquandalle