Skip to content
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

Support darkmode - Callouts 💬 #9277

Closed
Tracked by #9110
mxdvl opened this issue Oct 23, 2023 · 0 comments · Fixed by #9663
Closed
Tracked by #9110

Support darkmode - Callouts 💬 #9277

mxdvl opened this issue Oct 23, 2023 · 0 comments · Fixed by #9663
Assignees

Comments

@mxdvl
Copy link
Contributor

mxdvl commented Oct 23, 2023

We need to introduce a pattern for handling themes for Source kitchen components:

@mxdvl mxdvl self-assigned this Oct 23, 2023
@mxdvl mxdvl changed the title Support darkmode - Callouts Support darkmode - Callouts 💬 Oct 23, 2023
@DanielCliftonGuardian DanielCliftonGuardian added this to the DCR for Apps milestone Nov 30, 2023
@github-project-automation github-project-automation bot moved this to Triage in WebX Team Nov 30, 2023
@DanielCliftonGuardian DanielCliftonGuardian moved this from Triage to In progress in WebX Team Nov 30, 2023
mxdvl added a commit to guardian/csnx that referenced this issue Nov 30, 2023
## What are you changing?

- Refactor `Tabs` so it receives an optional `theme` object which is
used to define relevant custom properties which drive the colour of the
component
- Remove CSS Overrides as we enable consumers to override every single
colour
- Add stories to demonstrate usage
- Refactors `Tabs` with some destructuring, but this can be kept to a
different PR 0a75669

## Why?

- This enables us to use Callouts with dark mode in dotcom-rendering:
guardian/dotcom-rendering#9277
- Proposal for an answer to:
guardian/dotcom-rendering#9333
@github-project-automation github-project-automation bot moved this from In progress to Done in WebX Team Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants