Skip to content

Conversation

@rosselliott
Copy link
Contributor

If bootstrap.dropdown is already loaded then the multi select dropdown will never open as two handlers will be attached to the element. As the handlers will both toggle the 'open' class the net effect is that the dropdown will remain closed.

@JrmKrb
Copy link

JrmKrb commented Feb 8, 2017

Would be awesome if it could be merged!

@skratchdot skratchdot merged commit b2d01d2 into skratchdot:master Feb 9, 2017
@skratchdot
Copy link
Owner

@rosselliott / @JrmKrb - sorry for the delay on this. I missed it. Thank you for this PR!

@rvaidya
Copy link

rvaidya commented Feb 9, 2017

Can we get a version bump for this on npm?

@JrmKrb
Copy link

JrmKrb commented Feb 9, 2017

Thank you @skratchdot !

@skratchdot
Copy link
Owner

@rvaidya - the build isn't working for me right now. I'm going to try to make a few updates before publishing. Might finally get around to adding some tests as well. I probably won't get to this until after work today (so if I forget, feel free to drop another message in here)

sman591 added a commit to sman591/react-bootstrap-multiselect that referenced this pull request Apr 19, 2017
Fixes an issue introduced in skratchdot#61
sman591 added a commit to sman591/react-bootstrap-multiselect that referenced this pull request Apr 19, 2017
This reverts commit b2d01d2, reversing
changes made to 0f72727.
@sman591
Copy link
Contributor

sman591 commented Apr 19, 2017

require('bootstrap') is throwing an error for me rather than silently falling back to require('./bootstrap-multiselect.js'); - I'm building with Webpack & babel.

sman591 added a commit to sman591/react-bootstrap-multiselect that referenced this pull request Apr 20, 2017
This reverts commit b2d01d2, reversing
changes made to 0f72727.
sman591 added a commit to sman591/react-bootstrap-multiselect that referenced this pull request Apr 20, 2017
This reverts commit b2d01d2, reversing
changes made to 0f72727.
sman591 added a commit to sman591/react-bootstrap-multiselect that referenced this pull request Apr 24, 2017
This reverts commit b2d01d2, reversing
changes made to 0f72727.
sman591 added a commit to sman591/react-bootstrap-multiselect that referenced this pull request May 3, 2017
This reverts commit b2d01d2, reversing
changes made to 0f72727.
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.

5 participants