mobify / pinny

A mobile-first content fly-in UI plugin
MIT License
23 stars 4 forks source link

Should we provide a Pinny script prebuilt with all of the dependencies? #49

Open kpeatt opened 10 years ago

kpeatt commented 10 years ago

Using Pinny without using Require can be a little intense (even using Require can be a little intense). @MikeKlemarewski brought this up while trying to use it. Should we provide a prebuilt script with all the dependencies built in to make this easier on people who only want this functionality?

MikeKlemarewski commented 10 years ago

Thanks for raising this @kpeatt. I've also created another issue here: https://github.com/mobify/deckard/issues/3

That was the main sticking point I found...and didn't feel like battling through it. I think a prebuilt script with all the dependencies is a great idea if we're going to have this open sourced. If I saw at a flyout modal module with 8 script dependencies, I don't think I would consider using it, no matter how performant or pretty it is.

scalvert commented 10 years ago

@MikeKlemarewski agree. Bringing in 8 script dependencies is a bit crazy.

We could create a bundle via r.js. That is feasible.