Permalink
Browse files

Remove the cache-busting query parameters from the HTML. Details below:

In case you were wondering what ?v=2 was for.. read this:
http://html5boilerplate.com/docs/#Version-Control-with-Cachebusting

Query param revving is bad news:
http://www.stevesouders.com/blog/2008/08/23/revving-filenames-dont-use-querystring/

Even though that warning is from 2008, I've confirmed with Souders (a few months ago)
that this is still a concern.

The build script automatically revs filenames for you, so this is taken care of.
If you need to manually cachebust, then feel free to add the query params yourself, but
be aware of the small consequences. It shouldn't be a regular thing. ;)
  • Loading branch information...
1 parent 5b6db9d commit bb22ca66a8619808a87c1b5438845ed44baa4d3e @paulirish committed Mar 30, 2011
Showing with 6 additions and 6 deletions.
  1. +2 −2 demo/elements.html
  2. +2 −2 demo/tests.html
  3. +2 −2 index.html
View
@@ -26,10 +26,10 @@
<!-- CSS : implied media="all" -->
- <link rel="stylesheet" href="../css/style.css?v=2">
+ <link rel="stylesheet" href="../css/style.css">
<!-- Uncomment if you are specifically targeting less enabled mobile browsers
- <link rel="stylesheet" media="handheld" href="../css/handheld.css?v=2"> -->
+ <link rel="stylesheet" media="handheld" href="../css/handheld.css"> -->
<!-- All JavaScript at the bottom, except for Modernizr which enables HTML5 elements & feature detects -->
<script src="../js/libs/modernizr-1.7.min.js"></script>
View
@@ -26,11 +26,11 @@
<!-- CSS : implied media="all" -->
- <link rel="stylesheet" href="../css/style.css?v=2">
+ <link rel="stylesheet" href="../css/style.css">
<link rel="stylesheet" href="hack2.css">
<!-- Uncomment if you are specifically targeting less enabled mobile browsers
- <link rel="stylesheet" media="handheld" href="../css/handheld.css?v=2"> -->
+ <link rel="stylesheet" media="handheld" href="../css/handheld.css"> -->
<!-- All JavaScript at the bottom, except for Modernizr which enables HTML5 elements & feature detects -->
<script src="../js/libs/modernizr-1.7.min.js"></script>
View
@@ -24,10 +24,10 @@
<!-- CSS: implied media="all" -->
- <link rel="stylesheet" href="css/style.css?v=2">
+ <link rel="stylesheet" href="css/style.css">
<!-- Uncomment if you are specifically targeting less enabled mobile browsers
- <link rel="stylesheet" media="handheld" href="css/handheld.css?v=2"> -->
+ <link rel="stylesheet" media="handheld" href="css/handheld.css"> -->
<!-- All JavaScript at the bottom, except for Modernizr which enables HTML5 elements & feature detects -->
<script src="js/libs/modernizr-1.7.min.js"></script>

0 comments on commit bb22ca6

Please sign in to comment.