AMD Loading

Hey there,

Trying out your project to see if I can implement basic quill cursors on selection-change and text-change. I’m loading modules with RequireJS but am having difficulty with quill-cursors. I’ve listed Quill as a dependency for quill-cursors, however quill-cursors seems to be looking for Quill before instantiation of a Quill instance, which causes the following error.

quill-cursors.js:1804 Uncaught ReferenceError: Quill is not defined
    at Object.<anonymous> (quill-cursors.js:1804)
    at __webpack_require__ (quill-cursors.js:20)
    at Object.rangeFix (quill-cursors.js:75)
    at __webpack_require__ (quill-cursors.js:20)
    at Object.<anonymous> (quill-cursors.js:1817)
    at __webpack_require__ (quill-cursors.js:20)
    at Object.defineProperty.value (quill-cursors.js:66)
    at quill-cursors.js:69

My reasons for migrating to Webpack are piling up, so if this isn’t something you intend to address, maybe I should start the process of making a move.

Thanks for the great work

[Joeao]

Here’s a repo to reproduce the error. Kept it minimal for the sake of simplicity. If you have any questions stick them here or on the repo issues, it’s not serving any other purpose. Cheers guy

[pedrosanta]

Ok, nevermind what I said @Joeao, I’ve updated webpack configuration, because external config for Quill should have different ‘names’ depending on the module packaging/global var.

It should be working properly with your code now. Can you confirm and close the issue?