Add UMD #49

2015-02-12T11:52:00-08:00

alexanderGugel commented

2015-02-12 11:52:00 -08:00

(Migrated from github.com)

Make it work...... everywhere!

Putting computeLayout in the IIFE right away broke the transpiler for me. Nevertheless, I think using an out of the box UMD pattern makes sense here. I don't see a reason why css-layout shouldn't be script loader agnostic.

Make it work...... everywhere! Putting computeLayout in the IIFE right away broke the transpiler for me. Nevertheless, I think using an out of the box UMD pattern makes sense here. I don't see a reason why css-layout shouldn't be script loader agnostic.

vjeux commented

2015-02-12 12:41:55 -08:00

(Migrated from github.com)

Nice! Thanks

petehunt commented

2015-02-12 12:54:14 -08:00

(Migrated from github.com)

Hmm, would it be better to leave this package as CommonJS and treat umd as a build target?

vjeux commented

2015-02-12 13:21:10 -08:00

(Migrated from github.com)

@petehunt do you know what's the standard procedure for this?

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: DaddyFrosty/yoga#49