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

feat: Implement read/write of custom properties #111

Merged
merged 3 commits into from
Jan 23, 2020

Conversation

eps1lon
Copy link
Contributor

@eps1lon eps1lon commented Jan 23, 2020

Part of #89 which is blocking facebook/react#17896.

I did not implement full support since this requires more work and is only partially useful for JSDOM because resolution of values of the css variables requires the cascade.

Browser behavior: https://codesandbox.io/s/css-custom-properties-access-423ni

@jsakas
Copy link
Member

jsakas commented Jan 23, 2020

@eps1lon thanks for the PR! I think a lot of folks will be happy about this one.

I will merge and bump to version v2.2.0

@jsakas jsakas merged commit 26fbcec into jsdom:master Jan 23, 2020
@eps1lon eps1lon deleted the feat/custom-property-access branch January 23, 2020 15:57
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