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

Use HTML DOM style for setting label overflow as well #1143

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

adamchainz
Copy link

PR #928 fixed most inline style errors from Content Security Policy, but it missed this one. This fixes another error by again using style to add the style rule.

PR chartist-js#928 fixed most inline style errors from Content Security Policy, but it missed this one. This fixes another error by again using `style` to add the style rule.
@adamchainz
Copy link
Author

This is the error I was seeing on my application:
screen shot 2018-11-20 at 18 05 56

Building with this patch fixes it.

@dangreen dangreen force-pushed the develop branch 5 times, most recently from f818b93 to 3173275 Compare July 30, 2022 09:48
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.

None yet

1 participant