Skip to content

Fix for color-mode-theme mixin#1261

Merged
jonrohan merged 3 commits intomainfrom
color-mode-mixin-fix
Mar 26, 2021
Merged

Fix for color-mode-theme mixin#1261
jonrohan merged 3 commits intomainfrom
color-mode-mixin-fix

Conversation

@jonrohan
Copy link
Copy Markdown
Member

Fixing the color-mode-theme mixin. Currently the mixin doesn't take into account when mode="light" and light-theme="dark". This fix allows dark themes to be set as light themes and to toggle into single light mode.

/cc @primer/ds-core

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 26, 2021

🦋 Changeset detected

Latest commit: 739412b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@primer/css Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 26, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/primer/primer-css/7RrxqhE2GutMdF7inMfDBk79jeyy
✅ Preview: https://primer-css-git-color-mode-mixin-fix-primer.vercel.app

@vercel vercel bot temporarily deployed to Preview March 26, 2021 03:34 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants