Skip to content

Lowercase the doctype#111

Open
fulldecent wants to merge 3 commits into
mathiasbynens:masterfrom
fulldecent:patch-1
Open

Lowercase the doctype#111
fulldecent wants to merge 3 commits into
mathiasbynens:masterfrom
fulldecent:patch-1

Conversation

@fulldecent

Copy link
Copy Markdown

Replace every DOCTYPE to doctype.

Best practice identified at https://stackoverflow.com/questions/7020961/uppercase-or-lowercase-doctype#comment71796606_7020961


The achieves not only being the same size as the existing file in the repository, but, because of GZIP compression, this is likely to be smaller when sent across the network wire, up to possibly a FULL BYTE!

@dscorbett

Copy link
Copy Markdown
Contributor

This change adds an unnecessary newline.

@fulldecent

Copy link
Copy Markdown
Author

Fixed, thank you

@dscorbett

Copy link
Copy Markdown
Contributor

It is still there.

@fulldecent

Copy link
Copy Markdown
Author

Sorry, fixed again using file upload

@fulldecent

Copy link
Copy Markdown
Author

Requesting review again, please

Comment thread html5.html
@fulldecent

Copy link
Copy Markdown
Author

Golf bump here

@ross-spencer

Copy link
Copy Markdown

ecause of GZIP compression, this is likely to be smaller when sent across the network wire, up to possibly a FULL BYTE!

TIL! https://endtimes.dev/why-lowercase-letters-save-data/ (very cool)

@TPS

TPS commented Mar 13, 2025

Copy link
Copy Markdown

Maybe other web ML (xml, svg, ?) might benefit from careful application of this?

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

Successfully merging this pull request may close these issues.

5 participants