Skip to content
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

Add: index.html: meta tag #159

Closed
wants to merge 1 commit into from

Conversation

sotayamashita
Copy link

I added meta tag <meta http-equiv="x-ua-compatible" content="ie=edge">

ref:

http://stackoverflow.com/questions/6771258/what-does-meta-http-equiv-x-ua-compatible-content-ie-edge-do

I added meta tag `<meta http-equiv="x-ua-compatible" content="ie=edge">`
@ghost ghost added the CLA Signed label Jul 24, 2016
@lacker
Copy link
Contributor

lacker commented Jul 24, 2016

It seems like this is a choice that some developers but not most developers will want. From your stackoverflow link, without that tag, "IE will show the site how it thinks best it should be shown". That seems like a good default to me. So let's not add the <meta http-equiv="x-ua-compatible" content="ie=edge"> tag. It's easy enough to alter index.html if you do want to use a different IE-compatibility-mode.

@mxstbr
Copy link
Contributor

mxstbr commented Jul 24, 2016

Agreed @lacker 👍

Zhanks for taking the time to submit this PR @sotayamashita and kicking off this discussion, keep 'em coming! (though next time maybe as an issue first? 😉)

@mxstbr mxstbr closed this Jul 24, 2016
@lock lock bot locked and limited conversation to collaborators Jan 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants