Skip to content

Fix small typo in CSS variable name#369

Open
4levels wants to merge 1 commit intoHC200ok:mainfrom
4levels:main
Open

Fix small typo in CSS variable name#369
4levels wants to merge 1 commit intoHC200ok:mainfrom
4levels:main

Conversation

@4levels
Copy link
Copy Markdown

@4levels 4levels commented May 23, 2024

As explained in issue #367 this is a tiny commit that just adds the missing -- prefix for a CSS variable.

Type

What types of changes does your code introduce?
Fix a tiny typo in a CSS variable name, doesn't need a custom test imho.

Put an x in the boxes that apply_

  • Fix
  • Feature

Checklist

Put an x in the boxes that apply._

  • Title as described
  • Add unit test by vitest if necessary

As explained in issue [HC200ok#367](HC200ok#367) this is a tiny commit that just adds the missing `--` prefix for a CSS variable
@FahriDevZ
Copy link
Copy Markdown

Thank you for this fix 🙏

Just to add some context:
this typo causes a hard build failure when using lightningcss, since it
strictly validates CSS syntax and does not allow invalid var() declarations.

In my case, this completely blocks the build, so having this fix merged is
really important for projects that rely on lightningcss.

Appreciate the work on this PR — it would be great to see it merged soon.

@agattaollamast
Copy link
Copy Markdown

agattaollamast commented Mar 24, 2026

PLEASE merge this ASAP, same issue about "hard build failure when using lightningcss", this is used by default in vite v8, this issue is blocking build!

@YIllmann
Copy link
Copy Markdown

YIllmann commented Apr 8, 2026

@HC200ok

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.

5 participants