-
Notifications
You must be signed in to change notification settings - Fork 90
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
Improper CSS file inclusion technique #62
Comments
@yegor256 please, pay attention to this issue |
@rultor release, tag is |
@smartclash @yegor256 Oops, I failed. You can see the full log here (spent 1min)
|
@rultor release, tag is |
@smartclash @yegor256 Oops, I failed. You can see the full log here (spent 2min)
|
@yegor256 Okay, awesome that you've actually started using RawGit CDN. But still, you are doing some mistakes. Actually, the CDN caches the data for 10years. So, if you make a change, you will not get the changes reflected into the CDN. So, you must include the version tag in the URL so that you get the correct source. But, while developing you can remove the For more information on which URL to use in which way, please refer to https://github.com/rgrove/rawgit/blob/master/FAQ.md#whats-the-difference-between-development-and-cdn-urls |
@smartclash please, check my changes in d53a8c2 What do you think now? |
@rultor release, tag is |
@smartclash @yegor256 Oops, I failed. You can see the full log here (spent 2min)
|
@kt3k can you please help me here? I don't understand why it can't find |
@yegor256 Don't let the person downloading your project find the version. You do the hardwork. Replace the URL to https://cdn.rawgit.com/yegor256/tacit/0.8.0/dist/tacit-0.8.0.min.css. There are many other plug-and-play frameworks. So, people might not be intrested in finding the version and linking it. Better provide this link (latest CSS) : https://cdn.rawgit.com/yegor256/tacit/0.8.0/dist/tacit-0.8.0.min.css |
@yegor256
|
@rultor release tag is |
@smartclash @yegor256 Oops, I failed. You can see the full log here (spent 2min)
|
@rultor release tag is |
@kt3k thanks, it worked! |
@smartclash I didn't understand your comment. Both links you provided are exactly the same :( Can you please explain again? |
@yegor256 Well, no prob. I will make a PR soon for it :) |
@smartclash thanks! #64 is not technically correct, since it goes to |
@yegor256 I think the main purpose of this issue and usage of a CDN is done. I suggest we close this issue in particular and if @smartclash wishes, @smartclash can open an issue following-up comment #62 (comment) with more detail and even open PRs. |
Hey there @yegor256 and @filfreire, Sorry I was off from this PR and failed to contribute. Well, idea of @filfreire is good. We can make up a issue in that comment and close this issue. Okay? |
@smartclash as discussed above, I'm closing this issue. Feel free to open an issue following-up comment #62 (comment) with more detail and even open PRs. |
The job is not in WBS, won't close the order |
Hi,
I just came across your project and I find it a very good idea. The only problem is that you are violating the rules of GitHub. As per the github norms, we should directly include the CSS files to our project. But many still didn't obey.
To tackle this issue, GitHub served all the CSS files directly liked with header
Content-Type: text/plain
. So many modern browsers will understand it and will assume the CSS file to be a regular text file and thus, the whole page looks like a Ugly 1980's page.So, a great person named Ryan Grove created a service called RawGit which serves all the content with proper
Content-Type
headers so that browsers behave well. And as a plus feature, the production URL in the RawGit gets routed through MaxCDN to make it blazing fast.The main objective of this issue is to make you aware that you SHOULD NOT promote people to use this type of method. Instead there are many CDN providers which offer free CDN to open source project.
Check out :
The text was updated successfully, but these errors were encountered: