-
Notifications
You must be signed in to change notification settings - Fork 314
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
CDN files #47
Comments
Actually the whole thing needs a bit of configuring since the container/gutter size values are used throughout the stylesheet (not to mention your choice of whether to normalize, reset, or neither). |
Sorry, but I & most devs don't agree with including resets that you MIGHT use as part of the main CSS file. If you want people to edit text, IMHO it should not be inside the files you provide, but inside their own HTML, CSS, or JS. Same idea with your JS configuring; you don't want them to hack your main JS files. So many better & more flexible choices:
There are a few things you can do to help your users, so the main skel.css can be on CDN, without inducing configuring CSS:
|
I'll look into this. Thanks for the feedback. |
Welcome. Sorry I made you drink from the firehose ;) |
I was going to throw this up on a free CDN, jsDelivr, but saw the CSS is not minified, & seems that it needs configuring.
IMHO resets should be separate, & with jsDelivr you can d/l a concocted file. (eg
//cdn.jsdelivr.net/g/normalize@3.0,skeljs@0.5.0(skel-noscript.min.css)
). If there is configuring needed, then perhaps there should be separate files that you can load as-needed?The text was updated successfully, but these errors were encountered: