Skip to content

Commit

Permalink
Remove Google Frame message
Browse files Browse the repository at this point in the history
As said at http://blog.chromium.org/2013/06/retiring-chrome-frame.html Google Frame was retired/obsolete.

Changed "chromeframe" class to "browsehappy"
  • Loading branch information
Gustavo Rodrigues authored and paulirish committed Jun 24, 2013
1 parent 8f74370 commit 02713ee
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions css/main.css
Original file line number Diff line number Diff line change
Expand Up @@ -79,10 +79,10 @@ textarea {
}

/* ==========================================================================
Chrome Frame prompt
Browse Happy prompt
========================================================================== */

.chromeframe {
.browsehappy {
margin: 0.2em 0;
background: #ccc;
color: #000;
Expand Down
2 changes: 1 addition & 1 deletion index.html
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
</head>
<body>
<!--[if lt IE 7]>
<p class="chromeframe">You are using an <strong>outdated</strong> browser. Please <a href="http://browsehappy.com/">upgrade your browser</a> or <a href="http://www.google.com/chromeframe/?redirect=true">activate Google Chrome Frame</a> to improve your experience.</p>
<p class="browsehappy">You are using an <strong>outdated</strong> browser. Please <a href="http://browsehappy.com/">upgrade your browser</a> to improve your experience.</p>
<![endif]-->

<!-- Add your site or application content here -->
Expand Down

3 comments on commit 02713ee

@FlashJunior
Copy link
Contributor

Choose a reason for hiding this comment

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

what about <meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1"> in index.html?
that should be <meta http-equiv="X-UA-Compatible" content="IE=edge">, right?

@paulirish
Copy link
Member

Choose a reason for hiding this comment

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

Yes. https://developers.google.com/chrome/chrome-frame/ indicates that the x-ua-compatible hint should be removed.

Do you want to make a PR for this? Thanks

@FlashJunior
Copy link
Contributor

Choose a reason for hiding this comment

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

ok => #1396

Please sign in to comment.