search

powmedia / backbone.bootstrap-modal

Bootstrap Modal wrapper for use with Backbone.
MIT License
183 stars 97 forks source link

Fixed backdrop removal #54

Open ihortymoshenko opened 10 years ago

ihortymoshenko commented 10 years ago

Hello!

Bootstrap 3 no longer triggers the click event on the backdrop. This PR fixes the bug.

Thanks!

rejas commented 10 years ago

I'd change the first part (line 89-91) to 

            if (!this.$el.is($(event.target)) || !this.options.allowCancel) {
                return;
            }

Otherwise the option "allowCancel:false" could be circumvented by clicking on the backdrop