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

Make typogram diagrams work in dark mode #528

Merged
merged 1 commit into from
Dec 18, 2023
Merged

Conversation

tabatkins
Copy link
Contributor

@tabatkins tabatkins commented Dec 18, 2023

Since the typogram library explicitly uses black fill/stroke and a transparent bg, it fails badly when the spec is in dark mode. Ideally this would be fixed upstream, but for now this is a minimal fix that appears to catch everything necessary.


Preview | Diff

@w3cbot
Copy link

w3cbot commented Dec 18, 2023

npm1 marked as non substantive for IPR from ash-nazg.

Copy link
Collaborator

@npm1 npm1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this!

@npm1 npm1 merged commit 87e8ca6 into w3c-fedid:main Dec 18, 2023
2 checks passed
github-actions bot added a commit that referenced this pull request Dec 18, 2023
SHA: 87e8ca6
Reason: push, by npm1

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@npm1 npm1 mentioned this pull request Dec 18, 2023
github-actions bot added a commit to mattdanielbrown/WebID that referenced this pull request Dec 20, 2023
SHA: 87e8ca6
Reason: push, by pull[bot]

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

3 participants