ladybird icon indicating copy to clipboard operation
ladybird copied to clipboard

LibCore: Remove .xht from common_extensions in MimeType text/html

Open rcorsi opened this issue 7 months ago • 1 comments

Extension .xht was part of common_extensions for two MimeType array entries: application/xhtml+xml and text/html. Should only be part of one.

Duplicate .xht extension removed from text/html mime type and is now only part of application/xhtml+xml mime type.

Pull request as a follow-up to #4860.

As Andrew Kaster mentions here, an authoritative reference for .xht file is not easy to find.

Some mention of the extension and these MimeTypes, at the IANA mime type registry.

https://www.iana.org/assignments/media-types/application/xhtml+xml https://www.iana.org/assignments/media-types/text/html

This Mozilla guide does not include the .xht extension in this table, only makes reference to .xhtml, html, htm

https://developer.mozilla.org/en-US/docs/Web/HTTP/Guides/MIME_types/Common_types

xhtml html

Looking further online we have lots of discussion about html vs xhtml

https://www.w3.org/TR/2010/WD-html-markup-20101019/documents.html#mime-types https://www.sitepoint.com/community/t/html-or-xhtml/24939 https://www.w3schools.com/html/html_xhtml.asp https://stackoverflow.com/questions/1969290/what-is-the-difference-between-html-and-xhtml-extension-xhtml-is-a-markup-lan

I'm sure more can be found. I didn't look hard for these.

rcorsi avatar May 28 '25 23:05 rcorsi

@ADKaster can you please have a look at this followup pull request when you have a few minutes? Should I rebase to latest, or is that not needed until there is a merge conflict?

rcorsi avatar Jun 03 '25 14:06 rcorsi

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions!

github-actions[bot] avatar Jun 25 '25 02:06 github-actions[bot]