-
Notifications
You must be signed in to change notification settings - Fork 23
-
Notifications
You must be signed in to change notification settings - Fork 23
New release? #91
Comments
Nope. SassyJSON should not be used. |
Ha! Care to elaborate on that? We're actually finding it quite useful for living styleguides - exporting a config map (e.g. colors) from Sass to JSON allows the styleguide to e.g. iterate over the colors in a template and show a swatch for each, which isn't possible otherwise. Do you have a preferable solution in mind for that? If you don't have a better solution in mind, and your belief is that SassyJSON should not be used and thus you don't plan to maintain it, how would you feel about transferring maintainership? |
SassyJSON was an experiment. It is not meant to be used in production and has never meant to be used in production. Is it a JSON parser written in Sass, by a wannabe developer. This is a recipe for disaster. There is no way to make SassyJSON a good tool, no matter who maintains it. It cannot be done properly, and for good reasons. I recommend you use a clean Sass plugin, either through Ruby, or through Eyeglass or Sassport. |
Hmm. We're using only the JSON export, not import, which seems (relatively) more straightforward. No disasters experienced thus far, and styleguide generation is only marginally "production" :-) I don't think Eyeglass is helpful in this case (unless I'm missing something it does). It does seem like sassport might provide a workable alternative, though; I'll look into that. |
I would recommend extracting the export part (which is quite straightforward and easy to maintain indeed) for your own purposes. |
Thanks for the clarity, and recommendations! |
Just FYI, we have extracted just the export part here: https://github.com/oddbird/sass-json-export (not yet released on npm). |
Any planned schedule for an npm release? The recently-fixed quoting bug is a deal-breaker for use with libsass.
Thanks! (We're using this project in our recent work on living styleguides...)
The text was updated successfully, but these errors were encountered: