-
Notifications
You must be signed in to change notification settings - Fork 76
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
Extend does not escape attributes #73
Comments
We have a special Maybe there is space to "extend" the |
Yes, I did indeed use an I noticed that changing this line to
fixes the problem. However, we would only want to call |
Yes and adding this would break use cases for users who do not want escaping. |
I am using the following to update node labels in an
Escape
but the new label is not automatically escaped. Maybe there is a better way to update node attributes that I'm missing.
On the other hand, if this is a bug, I'd happy to try to fix it. At a first glance at the implementation, it does not look like the escaping logic is executed for any
Attrs
method.Thanks!
The text was updated successfully, but these errors were encountered: