search

powmedia / backbone.bootstrap-modal

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

RequireJS/AMD question #48

Open thadk opened 10 years ago

thadk commented 10 years ago

https://github.com/powmedia/backbone.bootstrap-modal/blob/bootstrap3/src/backbone.bootstrap-modal.js#L272

Wouldn't this:

return Modal;

be better and more AMD / RequireJS compliant than what we have?

Backbone.BootstrapModal = Modal;
infacq commented 9 years ago

1+