This repository was archived by the owner on Jul 8, 2025. It is now read-only.
Replies: 2 comments
-
I think a linter could be added from day one. I doubt it would be a hard one to code, @alexasselin008 is a pro. |
Beta Was this translation helpful? Give feedback.
0 replies
-
Is it still relevant? If so, I think it should be moved to a Github Discussion until a formal proposal is available. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Problem to solve
Orbit tokens are prefixed with
alias
when they refer to another token. While looking at others 99% of them don't use alias for token names. Tokens names if explicit enough shouldn't need alias in their name asbg-accent
intent is clear enough. Alias pollutes autocompletion tooHere's an example of other system token naming convention :
Idea
Remove alias in token names, documentation should be enough to expose tokens that should be used. If this becomes a problem a linter could be added in order to help with the usage of tokens.
Doing this would require some restructuration of token naming e.g. text color tokens don't have text-color in their name this could lead to weird token names
default
is not a good token nametext-color-default
is much better.Beta Was this translation helpful? Give feedback.
All reactions