Skip to content
This repository has been archived by the owner on Mar 14, 2024. It is now read-only.

content: Unfinished article and inconsistency in Learn HTML 003 Metadata #10284

Closed
owyaggy opened this issue Jul 25, 2023 · 1 comment
Closed
Labels
content update for issues that do not require new content (only for updates to existing content)

Comments

@owyaggy
Copy link

owyaggy commented Jul 25, 2023

What page(s) need to be updated?

https://web.dev/learn/html/metadata/#other-useful-meta-information

Why is this update needed?

The current instruction about the of the demo website indicates web manifests are a useful tool, but are not under the purview of this module, and as such the element is somewhat long and unwieldy. This tracks with the code block shown at the end of the page.

However, the following paragraph inconsistently states:
It's pretty long, but it's done. It would have been much longer, but you've summed up all the icons, short name, etc. in a manifest file, enabling you to omit many and tags,

Including the sentence fragment - "tags," is the end of a paragraph, so this content is clearly unfinished, as well as inconsistent.

Since web manifests are covered in the PWA series and relevant articles are already linked, it seems the solution is changing this penultimate paragraph of the article to state instead:

"It's pretty long, but it's done."

What's the deadline?

N/A

Is it a hard deadline?

No.

@owyaggy owyaggy added the content update for issues that do not require new content (only for updates to existing content) label Jul 25, 2023
@owyaggy owyaggy changed the title content: Unfinished article and inconsistency in Learn HTML 003 Metadata content: P2: Unfinished article and inconsistency in Learn HTML 003 Metadata Jul 25, 2023
@owyaggy owyaggy changed the title content: P2: Unfinished article and inconsistency in Learn HTML 003 Metadata content: Unfinished article and inconsistency in Learn HTML 003 Metadata Jul 25, 2023
@dstmarthe
Copy link
Collaborator

Thanks for raising this. We’re moving to a new platform and so we’ll fix this over there. I’ve moved the issue to our new tracker and you can see it at https://issuetracker.google.com/issues/297891065

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
content update for issues that do not require new content (only for updates to existing content)
Projects
None yet
Development

No branches or pull requests

2 participants