-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
PopoverContent not re-rendering #125
Comments
It's likely missing the rerender on prop update. Right now it checks if isOpen changed. Not an issue if the inner content is self contained, I think. Definitely need to fix this use case. Thanks for reporting! |
Fix reactstrap#125 Let the TetherContent rerender when any props change while it is showing. This allows the children to update dynamically without needing to hide and show the content to force the update.
Yeah, the TetherContent only updates the content when it hides/shows. It needs a check for if it is showing/there is a container/element when the props change and to rerender (for a reason other than show/hide) the subtree if the container is there (similar to how the modal updates). |
Fix #125 Let the TetherContent rerender when any props change while it is showing. This allows the children to update dynamically without needing to hide and show the content to force the update.
Fix published in 3.0.1 |
Updated Example pointing to 3.0.1: http://codepen.io/eddywashere/pen/Xjbkmx |
If you edit the props for PopverContent, and it's already open, it doesn't re-render the content until closed and opened again.
Take a look at the following codepen... just open one and let it sit for a few seconds...
http://codepen.io/zeroasterisk/pen/zKxgrJ
You can see in the console.log that the state --> props are updating, but the UI is not. If you close and re-open the popover, it shows the new/current props.
The text was updated successfully, but these errors were encountered: