-
Notifications
You must be signed in to change notification settings - Fork 283
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
Bloated CSS [unrelated to textual content] #7
Comments
The reason is that I copied the site from another that used that stuff. I'd welcome a patch that streamlined things! |
This commit adds an unnecessary file, |
Thanks, this is now merged and deployed! |
ComparisonOld filesCSS
HTML template
Total sizeFile size: 1933 + 1284 + 12058 + 668 + 3738 + 818 = 20499 bytes New filesCSS
HTML template
Total sizeFile size: 648 + 238 = 886 bytes ConclusionThe total size of the affected files was reduced from 20499 bytes to 886 bytes. — Mathias |
It looks like semver#7 introduces a single-file `main.css`, and makes the other css files unnecessary. This commit also deletes these file to reduce confusion (and repo size).
Is there a reason why you’re including
blueprint.css
andsyntax.css
? You’re not using any of these at the moment.If you do not plan on ever doing so, I would be happy to rewrite the CSS for you in 1 file (instead of 5) containing ± 20 lines (instead of, what, like 1000?). This would greatly reduce HTTP requests and file size, i.e. bandwidth.
Just let me know :)
Cheers
The text was updated successfully, but these errors were encountered: