-
Notifications
You must be signed in to change notification settings - Fork 114
Conversation
Included requirements | ||
===================== | ||
Global Pollution | ||
================ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Included requirements is not main enough for the readme. What is required here is global pollution.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed. If they care about libraries we use, they can check out package.json
window.ObjectHistory = require("object-history/history"); | ||
WYMeditor.EXTERNAL_MODULES = { | ||
ObjectHistory: require("object-history/history") | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Renamed this file because it doesn't globalify any more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead, it creates and populates WYMeditor.EXTERNAL_MODULES
.
Looks good. 👍 |
Closes #638 Less global pollution
Thank you!!! |
No description provided.