Skip to content

Conversation

thecrypticace
Copy link
Contributor

@thecrypticace thecrypticace commented Aug 26, 2025

Given this variant:

matchVariant(
  "foo",
  (value) => `&:is([data-foo='${value}'])`,
  {
    values: {
      DEFAULT: "",
      bar: "bar",
      baz: "bar",
    },
  }
)

We weren't listing foo-bar and foo-baz in IntelliSense. This PR fixes that.

@thecrypticace thecrypticace force-pushed the fix/match-variant-suggestions branch from 45bbf6b to 8ba6b6e Compare August 26, 2025 13:47
@thecrypticace thecrypticace changed the title Show suggestions for all matchVariant values Show suggestions for known matchVariant values Aug 26, 2025
@thecrypticace thecrypticace marked this pull request as ready for review August 26, 2025 13:58
@thecrypticace thecrypticace requested a review from a team as a code owner August 26, 2025 13:58
@thecrypticace thecrypticace enabled auto-merge (squash) August 26, 2025 14:21
@thecrypticace thecrypticace merged commit 8165e04 into main Aug 26, 2025
7 checks passed
@thecrypticace thecrypticace deleted the fix/match-variant-suggestions branch August 26, 2025 14:25
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