so this will change to #file-not-found

width and height attributes hold the "intrinsic width/height of the image in pixels". You only need to specify the numeric value (the rest is ignored).

also there should be no extra commas in the value strings.

Having a title here, adds an h1 heading to the page that doesn't look inline with the rest of the design.
(Not sure what the correct solution is for such cases.)

The problem here is that Pete will report it as an error (which a missing title would be normally).
A solution would be to style that <h1> so it disappears.

How about we allow an empty @title tag, which indicates that we explicitly do not want a heading?

SGTM 😃

won't that be error-prone? how about we allowed @title VOID instead?

It is not that error prone because it is very unlikely that someone would add a @title tag and forget to add a title.

The support for this landed a few hours ago.

I just realised though that this might not be necessary. If no title is provided via @title then the content is searched for the first h1 element.

So if @sjtrimble changes the file so that <h2>Page Not Found</h2> becomes <h1>Page Not Found</h1> then we can ditch the @title and @description tags from the file altogether.

Wouldn't that render the title (which is exactly what we are trying to prevent)?

As I understand it, we do want a heading (there is already a h2 there) but not at the top of the page. The thing about adding a title via the @title tag is that it automatically adds a h1 at the top of the page.

Oh, I see. Then, I agree this could be solved by removing the @title and changing the h2 to h1 (and styling it appropriately).

The problem here is that Pete will report it as an error (which a missing title would be normally).
A solution would be to style that <h1> so it disappears.

As we discussed, a lighter weight alternative is an MD icon such as seen in the network fail page PR #16139.

I would merge this now and wait to change it a later PR.

You can remove this @description as well :-)