Skip to content

Check if menu ref is not null#656

Closed
gaastonsr wants to merge 1 commit intoJedWatson:masterfrom
gaastonsr:fix-ie-blur
Closed

Check if menu ref is not null#656
gaastonsr wants to merge 1 commit intoJedWatson:masterfrom
gaastonsr:fix-ie-blur

Conversation

@gaastonsr
Copy link

This fix should go to the commit with tag v0.9.1.

This fix is related to #539. I couldn't narrow the exact error but my function handleInputBlur is called twice, one with this.refs.menu set to a div element and the other to null.

Tested on IE11.

@JedWatson
Copy link
Owner

@gaastonsr you mean this is a fix for the 0.9.1 version? or the new 1.0 version in Master?

Unfortunately we've got conflicts, If you can update your branch I'll merge it in.

@gaastonsr
Copy link
Author

This is a fix for 0.9.1 version. Because as far as I can see, master doesn't have the changes introduced by #539.

If you merge it with v0.9.1 you shouldn't have problems.

@gaastonsr
Copy link
Author

Closing this because I updated the wrong file... New PR is on #661.

@gaastonsr gaastonsr closed this Dec 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants