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