Skip to content
This repository was archived by the owner on Jul 31, 2026. It is now read-only.

chore: inline utility variants - #306

Open
farazcsk wants to merge 1 commit into
mainfrom
faraz/clean-up-tokens
Open

chore: inline utility variants#306
farazcsk wants to merge 1 commit into
mainfrom
faraz/clean-up-tokens

Conversation

@farazcsk

@farazcsk farazcsk commented Oct 6, 2025

Copy link
Copy Markdown
Contributor

@farazcsk
farazcsk requested a review from adaam2 October 6, 2025 11:47
@farazcsk
farazcsk force-pushed the faraz/clean-up-tokens branch from 9c01221 to 4d3e12c Compare October 6, 2025 11:48

@adaam2 adaam2 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is this a breaking change?

cursor[bot]

This comment was marked as outdated.

@farazcsk

farazcsk commented Oct 6, 2025

Copy link
Copy Markdown
Contributor Author

Is this a breaking change?

Nope!

Not unless the css variables were being used directly (which they shouldn't)

cursor[bot]

This comment was marked as outdated.

@adaam2

adaam2 commented Oct 6, 2025

Copy link
Copy Markdown
Member

Is this a breaking change?

Nope!

Not unless the css variables were being used directly (which they shouldn't)

Will need to check this as there might be some css modules but go ahead and we'll deal with it

@farazcsk
farazcsk force-pushed the faraz/clean-up-tokens branch 3 times, most recently from 88019af to 8345e2b Compare October 6, 2025 12:32
Removes the need to create a new variable for each util.
@farazcsk
farazcsk force-pushed the faraz/clean-up-tokens branch from 8345e2b to 69d235d Compare October 6, 2025 12:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants