Twenty Twenty: Custom Logo bug in the Customizer

[acosmin]

This ticket is based on a ​bug discovered by @ilovewpcom

 I think I found a weird bug in the theme. It's not breaking anything, just some odd behaviour.

 Steps to replicate the issue:
 1. Start with a website that doesn't have a logo in the Site Identity.
 2. Open the Customize page and open the Site Identity panel.
 3. Upload a logo and DON'T click the Publish button.
 4. Change the site title or description under the logo.

 Right after step 4 you will notice that the logo disappears in the previewer and the site title becomes a lot bigger.
 If you click the Publish button to save the changes then everything is correct on the front-end.
 The only issue is with the previewer's odd behaviour.
 I hope this helps :)

A solution for this is to add:

'container_inclusive' => version_compare( $GLOBALS['wp_version'], '5.5', '>=' ),

on class-twentytwenty-customize.php:L50

It probably needs more testing.

[sabernhardt]

Thanks for the report!

I reproduced the error in Twenty Twenty with several earlier WordPress versions, going back to the minimum version (4.7):

5.8
5.4.6
5.3.8
5.0.13
4.9.18
4.8.17 (with WP_DEBUG set to false)
4.7.21 (WP_DEBUG false again)

So this probably does not need to check the version, and container_inclusive seems to fix it.

Note: I did not experience the same problem with other bundled themes, from Twenty Fifteen to Twenty Twenty-One (using WP 5.8).

[Boniu91]

Working as expected.

Test Report

Env

• WordPress 5.9-alpha-51272-src
• Theme: Twenty Twenty and Twenty Twenty One

Steps to test

1. Use Twenty Twenty
2. Go to customizer choose Site Identity section
3. Upload logo
4. Change the Site Title
5. Play around with adding/removing logo, changing Title and Description in order to find other glitches

Expected behaviour:
Logo doesn't disappear

[chaion07]

Hi @acosmin! Thank you for reporting this. Since discussing this recently during a bug-scrub session we feel that this testing is sufficient. Our team would still test this to confirm and get this ready for Commit. Cheers!

[hellofromTonya]

Assigning to myself to prepare the commit.

[hellofromTonya]

In 52022:

Bundled Theme: Ensure logo displays in Customizer previewer.

Fixes an issue with the logo not displaying the Customizer previewer when changing the site title or description.

Follow-up to [46271].

Props acosmin, boniu91, sabernhardt.
Fixes #51337.

[hellofromTonya]

Thank you everyone for contributing. The fix has been committed and will ship with WordPress 5.9.