Skip to content

Conversation

@amitamrutiya
Copy link
Contributor

Notes for Reviewers

This PR fixes #

Signed commits

  • Yes, I signed my commits.

@amitamrutiya amitamrutiya merged commit 0fd31c7 into layer5io:master Feb 3, 2025
5 checks passed
default: string;
};
tabs?: string;
elevatedComponents?: string;
Copy link
Member

Choose a reason for hiding this comment

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

@amitamrutiya we need to sure of this.. we don't want to add more and more tokens based on our needs otherwise we will be doing same mistake as it was in meshery of having so many colors. And we should use more genric names for such.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This token #303030 we heavily used in Kanvas so I have to add it.

@amitamrutiya amitamrutiya deleted the add-token-new branch February 4, 2025 13:27
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