-
Notifications
You must be signed in to change notification settings - Fork 26
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
Support for attributes? #13
Comments
Alt is supported, since you can access it as a DOM Node property. vdom's |
Thanks for your quick reply @marcelklehr! I really want to get this combination to work so your help is really appreciated. Right now in the following example, the What I want to do is:
The code looks like
but it looks like the patched in the end does not have attributes. |
Can you post the virtualized vtree? vdom-virtualize should overtake all properties listed here |
How is it supposed to handle the MUST_USE_ATTRIBUTE attributes? I tried setting |
It doesn't handle MUST_USE_ATTRIBUTE atm, see https://github.com/marcelklehr/vdom-virtualize/blob/master/index.js#L107 |
Attributes are now supported! |
Hi @marcelklehr ,
Does vdom-virtualize support attributes? For example, I have
alt
attribute in my dom that contains crucial information that I can't leave inside the element.I saw your comment in #1. Is this a problem with
virtual-dom
or it is something that we can change by patching up this virtualizer?Thanks!
Ge
The text was updated successfully, but these errors were encountered: