Add the bird image to the homepage header

[kjellr]

Fixes #92.

This PR uses the new pattern block to place the header-large-dark pattern in the homepage's header. Like #182, this PR relies on WordPress/gutenberg#36090 to get it to appear correctly.

Before	After

[jffng]

Nice, love that we can remove repetitive code. Assuming the Gutenberg PR lands, this LGTM.

[kjellr]

@jffng just pushed a quick update to re-use the header.html template part (as per the method in #55, which I think we still need to use). Could use another review just to be sure. 👍

[carolinan]

To get the dark header, I selected the header template part and used replace.
The image of the bird is showing, but the dark header is not the same width as earlier. Only the last, the inner most group block, is set to full width. 🤔
Ah, that's why the GB pr was needed.

[carolinan]

Also, the preview is not working for the dark header. which needs to be an upstream issue, if one hasn't already been opened

[kjellr]

Also, the preview is not working for the dark header. which needs to be an upstream issue, if one hasn't already been opened

What are you seeing there? It did work for me, but it took a minute for the top of the header to appear:

[carolinan]

I mean this preview not the pattern preview in the sidebar:

[carolinan]

Who has time to wait a full minute? :) (waiting doesn't help for this preview...)

[carolinan]

The problem with the theme attribute is that if I move the theme into a subfolder like wp-content/themes/theme-experiments/, this message is shown:
Template part has been deleted or is unavailable: header

[kjellr]

I mean this preview not the pattern preview in the sidebar:

I'm already seeing errors with this preview in trunk, so I'm not sure it should hold up this PR:

The problem with the theme attribute is that if I move the theme into a subfolder like wp-content/themes/theme-experiments/, this message is shown:
Template part has been deleted or is unavailable: header

I moved the theme into a subfolder and things continued to work as expected.

Even so, I tried out an alternate approach that uses a pattern just for the text + bird image, but it requires more code and also showed preview errors. So I'm not totally sure it's worth it.

Would someone mind testing this out in a subfolder to see if it works or not? If it's still a bug, I can push that alternate approach for now.

[jffng]

This is what I see when I insert the dark header and the theme is a subfolder:

[kjellr]

Thanks for testing. I wonder why it worked for me. 🤔

Anyway, I pushed that second, more elaborate method. Would you mind giving that a try next?

[jffng]

Anyway, I pushed that second, more elaborate method. Would you mind giving that a try next?

This worked for me with the theme in a subfolder, no template part not found error.

[jffng]

Two small typo fixes and this LGTM.