search

devpunks / snuggsi

snuggsi ツ - Easy Custom Elements in ~1kB
https://snuggsi.com
MIT License
398 stars 17 forks source link

Consider a modular design of snuggsi #25

Closed grepsedawk closed 7 years ago

grepsedawk commented 7 years ago

Just something I wanted to bring up, but snuggsi is growing at an amazing rate! Things like the fileuploader that @VicenteRD is building could be seen as "Out of the original scope of snuggsi", and could add to the overall repository's size when it may be a non-essential piece. Have you considered going a modular snuggsi approach where you would have a base (let's say around 1kb) then adding literal "plugins" or "modules" to add additional functionality?

Just an idea I had floating around my head a bit the last day or two.

brandondees commented 7 years ago

yeah I think keeping it all in the main sandbox repo for now is ideal, but as it gets closer to "consumer ready" it'll be preferable to isolate the individual pieces in keeping with the "only take what you need" minimalistic spirit, and then just link out to all that stuff from the main documentation / examples repo

brandondees commented 7 years ago

the package pulled as an NPM dependency should really only be "the actual code you're using" and a little bit of surrounding metadata and documentation

snuggs commented 7 years ago

Might be able to get @cristhiandick help on the README as well.

True @brandondees only thing an npm install snuggsi would give them is the dev server. We can be more explicit about that. /cc @pachonk

snuggs commented 7 years ago

@pachonk may want to check out this issue as well. @brandondees created it and we have been using it as a north star with all the examples coming through. https://github.com/devpunks/snuggsi/issues/5

snuggs commented 7 years ago

Closing this @pachonk . As i think we have this sorted out. Clone repo and npm browse or yarn or whatever you use and off to the races.

http://snuggsi.herokuapp.com for online version. Also each major "library" in root has it's own README.md now.

grepsedawk commented 7 years ago

Yeah, this should be closed. Thanks