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

consider lowercase X-UA-Compatible #1656

Closed
BLooperZ opened this issue Jan 12, 2015 · 15 comments
Closed

consider lowercase X-UA-Compatible #1656

BLooperZ opened this issue Jan 12, 2015 · 15 comments

Comments

@BLooperZ
Copy link
Contributor

According to http://msdn.microsoft.com/en-us/library/jj676915.aspx:

The X-UA-Compatible header isn't case sensitive;

please consider making the meta tag from:
<meta http-equiv="X-UA-Compatible" content="IE=edge">
to:
<meta http-equiv="x-ua-compatible" content="IE=edge">
same reasons like Issue #1522

@arthurvr
Copy link
Member

please consider making the meta tag from:
<meta http-equiv="X-UA-Compatible" content="IE=edge">
to:
<meta http-equiv="x-ua-compatible" content="IE=edge">

Is there any reason you'd still keep the IE capitalised?

At first this looks good to me, but I'd like to do some manual testing first. Nobody wants it's page rendering as it was IE6 😄

@BLooperZ
Copy link
Contributor Author

In this source they have examples of using the meta tag,
you can notice that they do use it in lowercase though the "IE" stays capitilised,
so I don't know whether the "IE" statement is case-insensitive as well, need testing.

@battaglr
Copy link
Contributor

If http-equiv and content are both case insensitive I would give this a big 👍.

Update: if I'm not wrong gzip normally works better when using lowercase... @alrra usually knows better about this kind of stuff. :)

@ghost
Copy link

ghost commented Jan 26, 2015

I have tested this in IE 11 and yes this : <meta http-equiv="x-ua-compatible" content="ie=edge"> does work 👍 . I am happy to implement this if everyone is happy for this change to go ahead.

@arthurvr
Copy link
Member

I have tested this in IE 11 and yes this : <meta http-equiv="x-ua-compatible" content="ie=edge">

Tested in IE11? Starting with IE11, document modes are deprecated :). Please refer to https://msdn.microsoft.com/en-us/library/ie/bg182625(v=vs.85).aspx#docmode. More importantly this needs to be tested on older browsers.

@ghost
Copy link

ghost commented Jan 26, 2015

Oops completely forgot about that! Although I have reason to believe it will work as to check it was not just defaulting out at edge I also tried <meta http-equiv="x-ua-compatible" content="ie=8"> which resulted in rendering with document mode 8. However just to be on the safe side I will try with older versions of IE after school tommorow.

@ghost
Copy link

ghost commented Feb 2, 2015

Ok so I have tested this in IE 10 and it works with all lower case. I may be a will before I can test in IE9 and 8 as the vm's that Microsoft provide won't import into virtual box at the moment :(

@kevinSuttle
Copy link

@arthurvr
Copy link
Member

arthurvr commented Feb 8, 2015

I was able to do little testing myself and everything works 🎉. You want to PR this, @BLooperZ?

@BLooperZ
Copy link
Contributor Author

BLooperZ commented Feb 9, 2015

did you check it with lowercase "ie" as well?

@alrra
Copy link
Member

alrra commented Mar 4, 2015

Browser (OS) Screenshot
IE 8
(Windows XP)
IE 9
(Windows XP)
IE 10
(Windows 7)

Everything seems to work.

I'm not wrong gzip normally works better when using lowercase

@battaglr If you have more text in lowercase, then usually yes.

same reasons like Issue #1522

Haha

If anyone is interested in some stats:

(:warning: Please keep in mind that those numbers are not necessarily representative for actual pages with content, so treated them just as fun stats!)

  • Using <meta http-equiv="X-UA-Compatible" content="IE=edge">

    original size:         1820 B
    gzipped size:           885 B
    ─────────────────────────────
    reduction:              935 B [51.4%]
    

    Compression efficiency heat map:

  • Using <meta http-equiv="x-ua-compatible" content="ie=edge">

    original size:         1820 B
    gzipped size:           888 B
    ─────────────────────────────
    reduction:              932 B [51.2%]
    

    Compression efficiency heat map:

@alrra alrra closed this as completed in d8e4f16 Mar 4, 2015
alrra added a commit that referenced this issue Mar 4, 2015
alrra pushed a commit that referenced this issue Mar 4, 2015
Change made in order to be more consistent with the overall use of
lowercase in `index.html`.

- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -

From https://msdn.microsoft.com/en-us/library/jj676915.aspx:

  "The X-UA-Compatible header isn't case sensitive; however, it must
   appear in the header of the webpage (the HEAD section) before all
   other elements except for the title element and other meta elements."

- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -

Ref #1656

Close #1656
Close #1668
alrra added a commit that referenced this issue Mar 4, 2015
@battaglr
Copy link
Contributor

battaglr commented Mar 5, 2015

@alrra: great! off-topic: which tool do you use to generate those heat maps?

@patrickkettner
Copy link
Contributor

its gzthermal

@alrra
Copy link
Member

alrra commented Mar 5, 2015

off-topic: which tool do you use to generate those heat maps?

@battaglr Like @patrickkettner said, I've used gzthermal.

@battaglr
Copy link
Contributor

battaglr commented Mar 5, 2015

Thank you both!

mrmartineau added a commit to TryKickoff/kickoff that referenced this issue Sep 29, 2015
eleanor-byhook pushed a commit to eleanor-byhook/html5-boilerplate that referenced this issue Feb 29, 2016
Change made in order to be more consistent with the overall use of
lowercase in `index.html`.

- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -

From https://msdn.microsoft.com/en-us/library/jj676915.aspx:

  "The X-UA-Compatible header isn't case sensitive; however, it must
   appear in the header of the webpage (the HEAD section) before all
   other elements except for the title element and other meta elements."

- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -

Ref h5bp/html5-boilerplate#1656

Close h5bp/html5-boilerplate#1656
Close h5bp/html5-boilerplate#1668
eleanor-byhook pushed a commit to eleanor-byhook/html5-boilerplate that referenced this issue Feb 29, 2016
jeffreznik pushed a commit to jeffreznik/sw-test that referenced this issue Oct 15, 2017
Change made in order to be more consistent with the overall use of
lowercase in `index.html`.

- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -

From https://msdn.microsoft.com/en-us/library/jj676915.aspx:

  "The X-UA-Compatible header isn't case sensitive; however, it must
   appear in the header of the webpage (the HEAD section) before all
   other elements except for the title element and other meta elements."

- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -

Ref h5bp#1656

Close h5bp#1656
Close h5bp#1668
jeffreznik pushed a commit to jeffreznik/sw-test that referenced this issue Oct 15, 2017
itskingori added a commit to itskingori/minimal that referenced this issue Feb 3, 2018
Change made in order to be more consistent with the overall use of
lowercase in `index.html`.

From https://msdn.microsoft.com/en-us/library/jj676915.aspx:

  "The X-UA-Compatible header isn't case sensitive; however, it must
   appear in the header of the webpage (the HEAD section) before all
   other elements except for the title element and other meta elements."

Source: h5bp/html5-boilerplate#1656
alrra added a commit to alrra/dotfiles that referenced this issue Oct 12, 2020
cynthiabsmindlr added a commit to cynthiabsmindlr/code-guide that referenced this issue Aug 14, 2024
This is really minor, I know, but HTML5 Boilerplate changed this to all lower case. With Gzip compression on it can save 3 bytes. :-)
REF: h5bp/html5-boilerplate#1656
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants