add append hook

[sourcec0de]

see my PR for details #40

[amorey]

The append hook looks useful but the name feels ambiguous. Did you consider any other names? beforeAppend? insert?

[amorey]

@sourcec0de - do you have any thoughts on the naming convention? I'm leaning towards calling the attribute insert instead of append.

[sourcec0de]

@amorey I think that makes sense.
I can't think of a way to easily indicate that it's overriding the insertion. I kind of like beforeInsert

[amorey]

Ok, thanks for the feedback. One problem with beforeInsert is that it isn't clear that the method overrides the DOM insertion method.

[sourcec0de]

Sure, in that case i think append makes sense.

[sourcec0de]

Once it's decided would you like me to update my PR?

[amorey]

Sure, if you can update the PR that would be great. I thought you preferred insert to append...

[amorey]

Here's another idea - how about combining the before and append callbacks into one method? If the before callback returns false then we can bypass the default insertion method at L143:
https://github.com/muicss/loadjs/blob/master/src/loadjs.js#L143

If we decide to do it this way then I think beforeInsert would be an appropriate name.

[sourcec0de]

That's clean!!!!!

…

On Tue, Mar 21, 2017, 8:56 AM Andres Morey ***@***.***> wrote: Here's another idea - how about combining the before and append callbacks into one method? If the before callback returns false then we can bypass the default insertion method at L143: https://github.com/muicss/loadjs/blob/master/src/loadjs.js#L143 If we decide to do it this way then I think beforeInsert would be an appropriate name. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#41 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABgea2dNBOegA7oiiv1_Rh_7McjIQhTZks5rn_MbgaJpZM4MhWVE> .

[amorey]

Glad you like it! Do you want to modify the PR?

In terms of the name, maybe we should keep before for backwards compatibility.

[amorey]

@sourcec0de The latest version of LoadJS (v3.5.0) has support for this functionality. Just return false in the before callback to bypass the default DOM insertion mechanism:
https://github.com/muicss/loadjs
https://www.npmjs.com/package/loadjs

Let me know if you run into any issues using the new feature.