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

Update dialog window to update state open #424

Closed
wants to merge 2 commits into from
Closed

Update dialog window to update state open #424

wants to merge 2 commits into from

Conversation

oliviertassinari
Copy link
Member

componentWillReceiveProps is invoked when the component is receiving new props.

@oliviertassinari
Copy link
Member Author

This is exactly the point of this pull request. The openImmediately was added to toggle the window. I'm extending this behaviour.

@jkruder
Copy link
Contributor

jkruder commented Mar 18, 2015

Just to be clear, your use case is a dialog window that is not set to open immediately and then after some event (the dialog-window is already mounted at this point) you want to set openImmediately open the dialog immediately?

@oliviertassinari oliviertassinari deleted the patch-6 branch June 3, 2015 20:03
@zannager zannager added the component: dialog This is the name of the generic UI component, not the React module! label Mar 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: dialog This is the name of the generic UI component, not the React module!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants