-
-
Notifications
You must be signed in to change notification settings - Fork 315
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Design Token] - Semantic Colors #1024
Comments
There's 2 problems with that:
In a simple, perfect world where you don't want or need to much customization, Skeleton should already handle that for you. I'd be curious what your problem with the current design system is and where you feel it hinders you in a way that makes you want a more context-aware or in my opinion context-assigning system, if you'd care to elaborate. |
I think what you're saying regarding the default components makes sense. Based on that, I would say that the documentation confuses me a bit: You're explaining here that the "recommended manner of styling most components" is
But in the components, I mostly found the Variants styling approach you probably referenced in your post. So it feels that the I realized the The That brings me probably to the main problem of my confusion and why I suggested the enhancement: A lot of documents talk about the color design tokens (too much noise) and in different ways: Theme Generator Streamlining this into a "80% of your use cases" and "The full flexibility and extensibility" section could eventually help to navigate a new user through the docs while keeping an easy-to-take learning curve. |
That's excellent feedback for the currently ongoing documentation overhaul: #783 |
This has come up in the past, unfortunately it's not a feature request but a whole new library. It completely invalidates and removes the majority of the mechanisms for how Skeleton operates. I understand there's a place for this, but it's not in Skeleton.
FYI we're in the midst of some documentation updates and I think the wording for this will need to change. This is now the recommended method for OVERRIDING styles locally. If you need a one-off change, this is a simple way to do that. If you need global/sweeping changes our recommended approach is to use our Selector classes and use global stylesheet overrides, per:
Variants are a rather late addition to the library, so they're not quite featured as prominently. I agree though, they're quite useful and should get more attention. This will be changing soon, with them receiving their own dedicated documentation page alongside design tokens before the next release.
Very strange choice of words here - one of the most critical and useful features of the library is noise? I'm sorry but unless I'm misunderstanding you that's objectively wrong. I might suggest watching the video we shared by Brad Frost that explains what design token are and perhaps lean more about how they operate: The features and mechanism of Skeleton come from a real need and satisfy real problems. It's ok if Skeleton isn't working the way you personally want it, but I can't help but feel our goals and your goals aren't quite aligned here. And again that's fine. If Skeleton doesn't operate the way you want it you have a few options:
Thanks for your time and input. Unfortunately there's very little actionable here that aligns with the project goals. If you wish to continue this conversation please create either a GitHub Discussion or reach out on Discord. |
@endigo9740, maybe we have a miscommunication here. If I'm talking about too much "noise", then I mean that your current state of documentation was hard for me to understand because skeleton has a lot of pages that talk about the same stuff in different ways (Different ways = harder to learn/understand). (See the links that I shared) At least I have the feeling because I'm new to Skeleton. So my whole point was about: Preciser Docs My initial suggestion was to replace the numeric with a semantic system. Why? I worked with both and found that semantics is simpler to use while covering most use cases. Nevertheless, as Sarenor already pinpointed: You also have a simplification called
So I really hope that the first part of our conversation was simply a misunderstanding and this is resolving the "Very strange choice of words" offense. |
Hey I get it. That's why our current and ongoing focus is improvements to our documentation. It's the last major hurdle before we consider this production ready. We've been building up the the current docs over the course of a year and in that time you're going to have creeping issues, inconsistently, and mismatched tone and direction. There's just no avoiding it in a project of this scope and scale. But this is exactly what we're aiming to improve here: To catch you up - we have a few technical prerequisites before we begin our content-focused improvements, which are slated to begin next week. I'll be generating a general guidelines for technical changes (migrate to the preview component, make sure prop values are accurate, etc) while also providing recommendations in terms of tone in order to provide a more consistent voice. Then we have a number of volunteers that will help implement this. I'll be personally overseeing the updates to the "Docs" section of Skeleton, which covers critical areas such as the introduction, onboarding, and educating our users on what Skeleton is and how to take advantage of it's full feature set. Keep in mind not every feature currently present was available a year ago - which is why there's gaps and inconsistently in direction. We can only address this now that we have the entire roster of features in front of us.
The thing you have to understand here is you're coming in at the 11th hour and suggesting a change that is widely out of scope for our current project and goals. I'm sure you mean well, but I've worked with systems that operate like this (ex: Angular Material) and it was was awful. It's limited and restrictive in ways many folks won't recognize until it's too late. Skeleton was purpose built to go down a different path. Additionally, this is not a trivial change in direction - this is a completely different approach to the foundation of the library and it's core tenants. To speak plainly here, it's highly unlikely to ever happen. Given this, the summary here is you're suggesting changes we're already aware of and in progress making, as well as a suggestion that does not align the the goals of our project. Hopefully that clears up any miscommunication on your end. Again I'd advise this discussion continue in another location - this isn't the place for this kind of discourse. |
Describe what feature you'd like. Pseudo-code, mockups, or screenshots of similar solutions are encouraged!
Skeleton uses a color palette design system consisting of values from 100 to 900:
https://www.skeleton.dev/elements/tokens
https://www.skeleton.dev/docs/generator
Instead, I would like to bring up Sematic Color Design Tokens.
This approach is more aware of the context (like background, border, foreground), making it easier for the user to achieve consistent theming throughout the app.
What type of pull request would this be?
Enhancement
Any links to similar examples or other references we should review?
https://reshaped.so/content/docs/tokens/color
The text was updated successfully, but these errors were encountered: