-
Notifications
You must be signed in to change notification settings - Fork 20
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
docs: dark mode #2054
base: main
Are you sure you want to change the base?
docs: dark mode #2054
Conversation
|
✅ Deploy Preview for red-hat-design-system ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Size Change: +569 B (+0.27%) Total Size: 208 kB
ℹ️ View Unchanged
|
Opinion here: I feel like in order to make this really viable we need full theming css layer. Instead of the switch swapping between |
I'm not entirely sure what you mean by full theming css layer, so let me describe what I think you mean, and respond to that. If I got it wrong, let me know. The way most sites do this is by toggling a class name or attribute on the body, for example shoelace uses So if we were to do such a thing, we'd add some hypothetical While that's a good approach for most sites, it runs contrary to the principles of theming in RHDS. In our design system, theme and color palette are two different things. When discussing RHDS, "theme" means customizing one or more color palettes. When we think about it that way, what need does the design system docs site have to customize the theme, outside of pages which provide examples of how to do so? Wouldn't that be confusing, a bait-and-switch? Rather, we should strive to make ux-dot an exemplar of our HTML-first approach to design systems engineering. In most cases the user of our design system (i.e. the page author) should get most of the utility of the DS from HTML tags and attributes. Occasionally, they may use CSS to customize designs (e.g. for a particular pattern or to implement a custom theme) and rarely they should use JavaScript APIs to provide interactivity (e.g. to toast an alert). The "big idea" of dark mode / theming / color palettes in RHDS is that authors who just want to use the sane defaults shouldn't have to write CSS or JS to get them. |
What I did
Screencast.From.2024-11-18.08-02-16.mp4
Testing Instructions
Notes to Reviewers
icons and ux of the switch are TBD