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

Set an attribute on the document in addition to the class #219

Merged
merged 2 commits into from
Apr 3, 2020

Conversation

ivoilic
Copy link
Contributor

@ivoilic ivoilic commented Mar 8, 2020

Currently only a class is added to the document. This makes it difficult if you are using a framework which overwrites your classes. This mirrors the solution that is already in place for this same issue. See issue #179.

Currently only a class is added to the document. This makes it difficult if you are using a  framework which overwrites your classes. This mirrors the solution that is already in place for this same issue.
@yoavweiss
Copy link
Contributor

@ivoilic Can you please join the WICG to appease the IPR bots?

@ivoilic
Copy link
Contributor Author

ivoilic commented Mar 9, 2020

@yoavweiss I've joined WICG and updated the PR due to a typo. Sorry for not doing so before I submitted the PR.

@yoavweiss
Copy link
Contributor

@ivoilic no worries :) The IPR bots are still not happy. Did you link your GH account with your W3C account?

@ivoilic
Copy link
Contributor Author

ivoilic commented Mar 9, 2020

@yoavweiss Yes, I just forced it to check again and it looks like it went through!

@ivoilic
Copy link
Contributor Author

ivoilic commented Mar 28, 2020

Any chance of getting this merged? This is a change that shouldn't have a negative impact on existing user and would be of great benefit to many.

@robdodson robdodson merged commit 0f5fc6c into WICG:master Apr 3, 2020
@robdodson
Copy link
Collaborator

published a v5.1.0 release with this in it. thank you!

@ivoilic
Copy link
Contributor Author

ivoilic commented Apr 3, 2020

@robdodson 🎉 Great! Although I just realized I probably should have updated the docs in this PR as well.

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