-
Notifications
You must be signed in to change notification settings - Fork 94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
2.0 #76
Comments
It's hard to give an ETA or even a featureset at this point, because we're still at the philosophical stage of asking ourselves what Kickstrap 2 (K2) is supposed to be. But here's where my head is:
|
Thanks for taking the time to type all that out! Those all sound like nice improvements to me. Below I typed out some thoughts I had while reading. I understand that you have put a lot of thought in this and I haven't (yet) so it could be this is all old news, or even bad suggestions, but in that case I would love to hear the explanation behind the choices and be a little smarter for the future :) On the easy for beginners, powerful for experts point: On the decoupled point: On the honest point: |
K2 reaaaally needs to get more RoR friendly, so I think I need an advisor on that end (points to you inquisitively?) In the 2.0 branch, I have three main folders:
|
Definitely want to group certain settings under "development" and "production" I'm actually just now starting to get that to work, if you look at the latest on 2.0. It's still buggy though and I'll have to figure out how to get it to "come in" at the right time. That is, some of the code requires values from settings to come in afterward (to override) and other parts require it to come in before (to provide variables) there is probably a workaround for js settings but less will be tricky. Great idea on making the directories customizable. I think this is a lesson learned in 1.x, by supposedly supporting different frameworks, K2 should acknowledge that they have their own unique directory structures. |
https://github.com/ajkochanowicz/Kickstrap/tree/2.0/proto/kickstrap Maybe settings.less and settings.js should be moved outside the KS folder? The user could then set the directory of Kickstrap relative to the settings file. |
https://github.com/ajkochanowicz/Kickstrap/blob/2.0/proto/kickstrap/settings.js Is this what you mean in terms of custom settings for the directories (there would be more than this, of course). |
Oh how frustrating I don't have the time to help you develop kickstrap further right now, all I can do right now is play the role of advisor (more than happy to do so btw). At this time I am developing a new (my first) application in rails but I can only work on it in my free time so I'm still at the beginner stage of learning ruby and rails. So far I have been a python/django developer and my day to day work is still in django. I will continue to work on the kickstrap gem since I will be using that in my project, also I think this is the most developer friendly way to make kickstrap available for rails, so let's focus on the gem for rails. A thought just popped up in my head: a pip package for django would be awesome as well. But in django it is not possible to add static files through pip packages afaik (never seen such a package at least) but it could provide some template tags/management commands to make things easier. I'll take a look at this in detail later on, see if it is possible and has added value over manually doing stuff. I'll think about the settings file some more, in an ideal situation it would work just like the settings file in django, however that is technically not possible in this case I think... I'll come back to you on this. Setting the directories like that is what I had in mind yes :) |
Ha, no problem. I understand, and I'm a django dev myself. Technically, all of Kickstrap is static for django. Getkickstrap.com, built in Django and Kickstrap, uses "/static/" as its rootDir, so I would probably just advise folks use npm's volo to install it. Adam Kochanowicz Le mercredi, mars 13, 2013 à 10:24 AM, Allard Stijnman a écrit :
|
Continued from here
To see progress on 2.0, fork the repo and switch to the "2.0" branch or view the files here
The text was updated successfully, but these errors were encountered: