knockout-jqueryui icon indicating copy to clipboard operation
knockout-jqueryui copied to clipboard

Please add support for AMD dependencies, don't expect dependencies to be provided via `this`

Open blmille1 opened this issue 7 months ago • 0 comments

I'm trying to integrate knockout-jqueryui in my webpack configuration. knockout-jqueryui is calling its factory with this, which is not webpack compatible.

Will you please consider updating the header of the library to check for AMD/CommonJS support and bring in your dependencies?

E.g., kind of like this... !function(a,b){"use strict";"object"==typeof exports?b(exports,require("jquery"),require("ko"),require("jqueryui")):"function"==typeof define&&define.amd?define(["exports","jquery","ko","jqueryui"],b):b(a.kojqui={},a.jQuery,a.ko)}

blmille1 avatar Jun 26 '25 16:06 blmille1