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 missing <title> element #249

Merged
merged 1 commit into from
Jul 5, 2024
Merged

Add missing <title> element #249

merged 1 commit into from
Jul 5, 2024

Conversation

Jayman2000
Copy link
Contributor

@Jayman2000 Jayman2000 commented Jul 4, 2024

Checklist

  • I have read the Contributor Guide
  • I have read and agree to the Code of Conduct
  • I have added a description of my changes and why I'd like them included in the section below

Description of Changes

According to the HTML Standard:

The title element is a required child in most situations, but when a higher-level protocol provides title information, e.g., in the subject line of an email when HTML is used as an email authoring format, the title element can be omitted.

Before this change, the default about.hbs file didn’t have a <title> element even though a <title> element is required in most situations. This change adds a <title> element in order to help make sure that the HTML generated by cargo-about is valid.

Related Issues

According to the HTML Standard [1]:

> The title element is a required child in most situations, but when a
> higher-level protocol provides title information, e.g., in the subject
> line of an email when HTML is used as an email authoring format, the
> title element can be omitted.

Before this change, the default about.hbs file didn’t have a <title>
element even though a <title> element is required in most situations.
This change adds a <title> element in order to help make sure that the
HTML generated by cargo-about is valid.

[1]: <https://html.spec.whatwg.org/multipage/semantics.html#the-head-element>
@Jayman2000 Jayman2000 requested a review from Jake-Shadle as a code owner July 4, 2024 23:35
@Jake-Shadle Jake-Shadle merged commit 7b08ebe into EmbarkStudios:main Jul 5, 2024
1 check passed
@Jayman2000 Jayman2000 deleted the title-element branch July 5, 2024 18:50
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.

2 participants