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

Make x-ua-compatible lowercase #16723

Closed
wants to merge 1 commit into from
Closed

Make x-ua-compatible lowercase #16723

wants to merge 1 commit into from

Conversation

coliff
Copy link
Contributor

@coliff coliff commented Jun 29, 2015

This is really minor, I know, but HTML5 Boilerplate changed this tag to all lowercase. With Gzip compression on it can save 3 bytes. :-)
REF: h5bp/html5-boilerplate#1656

This is really minor, I know, but HTML5 Boilerplate changed this tag to all lowercase. With Gzip compression on it can save 3 bytes. :-)
REF: h5bp/html5-boilerplate#1656
@cvrebert cvrebert changed the title Update template.html Make x-ua-compatible lowercase Jun 29, 2015
@cvrebert
Copy link
Collaborator

For consistency, this would need to be updated everywhere in the code, not just in this one example.
Personally, I don't think it's worth the effort or the resulting inconsistency with other third-party docs.

@cvrebert
Copy link
Collaborator

cvrebert commented Jul 5, 2015

@twbs/team Anyone have an opinion here? I'm -1 on account of inconsistency with Microsoft's docs and HTTP header capitalization conventions, and how tiny the size benefit is.

@patrickhlauke
Copy link
Member

I have a purely personal/aesthetic preference for lowercase, for no actual hard reason though.

@XhmikosR
Copy link
Member

XhmikosR commented Jul 5, 2015

I'm against this because I find it easier to read when it's uppercase. The same applies to
doctype too.
On Jul 5, 2015 11:54 AM, "Chris Rebert" notifications@github.com wrote:

@twbs/team https://github.com/orgs/twbs/teams/team Anyone have an
opinion here? I'm -1 on account of inconsistency with Microsoft's docs and
HTTP header capitalization conventions, and how tiny the size benefit is.


Reply to this email directly or view it on GitHub
#16723 (comment).

@hnrch02
Copy link
Collaborator

hnrch02 commented Jul 5, 2015

👎

@mdo
Copy link
Member

mdo commented Jul 6, 2015

Not worthwhile for v3 given it's a trivial change.

@mdo mdo closed this Jul 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants