-
Notifications
You must be signed in to change notification settings - Fork 274
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
Map colours from the colour palette from dark theme to light theme #4907
Comments
I'm afraid it won't work because of contrasts between foreground and background colors. For exemple, a It would need to introduce a level of abstraction indicating the semantic of the colour (page background, button background, button foreground, etc) |
@feloy AFAIK it should work fine to shape the initial design I mean, for example based on some guidelines we don't use charcoal for buttons, and bg- mean background, for text colour it would be text- something Of course it would require to review the final screen, but it can give a clear overview. Also the palette is quite limited so it could work. I think you're asking too much details, if everything need a context |
This would be a good topic for UX call, although due to US holiday we don't have one this week. I think there will be some rough mapping (and we'll have to extrapolate in order to apply light mode everywhere without UX being a bottleneck), but I agree with @feloy that it's unlikely to be a direct mapping for everywhere/all components. Maybe we can start by asking for a mockup of one or two screens (e.g. Dashboard or Containers page and Settings > Resources)? That'll give us something concrete to work on, and most of the mapping to apply elsewhere? |
Except the Settings button in the first page which is not contrasted, as a light mode guy, I like it :) 👍 |
nice progress Probably we would need to run screens with a11y tools checking contrast, etc |
FYI I added some comments there #5878 |
I notice that colors in penpot are different than the one currently implemented for the dark theme in penpot, hover is purple, current implementation is gray current implementation background color of navbar is charcoal-800 you confirm the new colors for dark theme that will replace the current implementation ? |
make the palette color being more embeddable elsewhere or reusable it is like a no-op change related to containers#5878 containers#4907 Signed-off-by: Florent Benoit <fbenoit@redhat.com>
make the palette color being more embeddable elsewhere or reusable it is like a no-op change related to containers#5878 containers#4907 Signed-off-by: Florent Benoit <fbenoit@redhat.com>
make the palette color being more embeddable elsewhere or reusable it is like a no-op change related to containers#5878 containers#4907 Signed-off-by: Florent Benoit <fbenoit@redhat.com>
I know @mairin is working on the official dark theme colouring - so i would say wait maybe until it's confirmed that it's complete. |
For the light theme, I have some updates. To add more contrast, I added in the use of purple for the second menu and upped the gray for the boxes and inverted background. I have checked all the text and icon contrast to make sure they comply with WCAG guild lines, everything is at least AA rated I believe. The only thing I'm worried about is the little panel of coloured icons in the main nav bar - I'll have to find a colour that would work with those. |
Another update. I added light mode into the theming section of the design system penpot doc. Here are the values:
I'm sure there'll be more components that come up that need converting so let me know if you come across any :) |
Is your enhancement related to a problem? Please describe
To implement light theme, we would need a mapping between colours of the current colour palette
like
bg-charcoal-800
needs to be mapped tobg-gray-200
Describe the solution you'd like
Defines a mapping between colours when using light or dark theme
Describe alternatives you've considered
any other suggestions is welcome
Additional context
No response
The text was updated successfully, but these errors were encountered: