Browse files

new modernizr.

  • Loading branch information...
1 parent 080026e commit f8f347e392489282d96286e91c2795ef741e752b @paulirish paulirish committed Feb 19, 2012
Showing with 6 additions and 7 deletions.
  1. +1 −2 404.html
  2. +1 −1 index.html
  3. +0 −4 js/libs/modernizr-2.5.2.min.js
  4. +4 −0 js/libs/modernizr-2.5.3.min.js
@@ -40,5 +40,4 @@ <h1>Not found <span>:(</span></h1>
<script src=""></script>

kblomqvist Feb 19, 2012


This must be a mistake?


nimbupani Feb 19, 2012


No, just fancy.


paulirish Feb 19, 2012


They're optional in HTML. This validates and all browsers have no problem with it.

Since everyone stole the fun out of 404.index I added the littlest bit of fun back.


djtriptych May 30, 2012

Fancy, but just kinda wasted my time trying to figure out if it was an error or not.

My guess is It looks wrong to most developers. Wasting dev time makes me sad.

Also, it's inconsistent within the project since index.html retains the closing tags.

Also, browsers aren't the only things that consume HTML.

@@ -27,7 +27,7 @@
<!-- All JavaScript at the bottom, except this Modernizr build.
Modernizr enables HTML5 elements & feature detects for optimal performance.
Create your own custom Modernizr build: -->
- <script src="js/libs/modernizr-2.5.2.min.js"></script>
+ <script src="js/libs/modernizr-2.5.3.min.js"></script>
<!-- Prompt IE 6 users to install Chrome Frame. Remove this if you support IE 6.

Some generated files are not rendered by default. Learn more.

Oops, something went wrong.
Oops, something went wrong.

0 comments on commit f8f347e

Please sign in to comment.