Skip to content

Commit

Permalink
Update Modernizr to remove Respond. Change placeholder Media Query.
Browse files Browse the repository at this point in the history
Previous version of Respond was known to cause problems in IE8.
It was not necessary for developers who chose to write their CSS
in a 'mobile first' manner. Furthermore, there are some concerns
about the performance penalty Respond imposes upon legacy versions
of IE. If a future version of Respond is shown to be performant
and more reliable, it may be reconsidered for inclusion.

Along with this change is a simplification of the placeholder
Media Queries. Now there is only 1 Media Query example and it is
more clearly labelled as an example. The use of `em` units in the
condition is to reinforce the idea that Media Query breakpoints
should not be device-specific.

Fix #816
  • Loading branch information
necolas committed Jan 24, 2012
1 parent f0358b5 commit af98e6f
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 13 deletions.
16 changes: 5 additions & 11 deletions css/style.css
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* HTML5 Boilerplate
* HTML5 Boilerplate
*
* What follows is the result of much research on cross-browser styling.
* Credit left inline and big thanks to Nicolas Gallagher, Jonathan Neal,
Expand Down Expand Up @@ -221,19 +221,13 @@ td { vertical-align: top; }


/* ==|== media queries ======================================================
PLACEHOLDER Media Queries for Responsive Design.
These override the primary ('mobile first') styles
EXAMPLE Media Query for Responsive Design.
This example overrides the primary ('mobile first') styles
Modify as content requires.
========================================================================== */

@media only screen and (min-width: 480px) {
/* Style adjustments for viewports 480px and over go here */

}

@media only screen and (min-width: 768px) {
/* Style adjustments for viewports 768px and over go here */

@media only screen and (min-width: 35em) {
/* Style adjustments for viewports that meet the condition */
}


Expand Down
Loading

1 comment on commit af98e6f

@kaidez
Copy link

@kaidez kaidez commented on af98e6f Jan 25, 2012

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Definitely the right thing to do here!

Please sign in to comment.