Skip to content

Conversation

@farazcsk
Copy link
Collaborator

@farazcsk farazcsk commented Oct 6, 2025

@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
Copy link
Member

@adaam2 adaam2 left a comment

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
Copy link
Collaborator Author

farazcsk commented Oct 6, 2025

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
Copy link
Member

adaam2 commented Oct 6, 2025

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 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.

3 participants