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
Re-render after tagStyles changed #366
Conversation
this works with the code:
Thank you so much! |
@Exilz LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nicely done! See my comments.
@@ -84,7 +84,7 @@ export default class HTML extends PureComponent { | |||
} | |||
|
|||
componentDidUpdate(prevProps, prevState) { | |||
const { html, uri, renderers } = prevProps; | |||
const { html, uri, renderers, tagsStyles } = prevProps; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you apply the same logic for classStyles
? Would fix #377
@jorgemasta I am going to rework and merge this one manually. Thanks for your contribution. |
Thanks @jsamr ! I was trying to find time to carry out your recommendation |
@jorgemasta I'm planning for a minor release soon, that is why :-) Plus, I started to implement some tests, but we don't provide guidelines yet. I need to implement a few "exemplar" tests so that contributors can have a template. If you're up to make the change I requested soon, let's say before Friday, I can wait for you. Either cases, I'll add the tests. |
This PR has been manually merged into master with commit 6e2ed46. |
Re-call
this.parseDOM
if thetagStyles
changed.Resolves #343