[wp-trac] [WordPress Trac] #61766: Multiple bundled themes: Add site logo theme support

[WordPress Trac]

#61766: Multiple bundled themes: Add site logo theme support
-------------------------------------------------+-------------------------
 Reporter:  poena                                |       Owner:  (none)
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  Awaiting
                                                 |  Review
Component:  Bundled Theme                        |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  changes-requested has-patch has-     |     Focuses:
  screenshots                                    |
-------------------------------------------------+-------------------------
Changes (by ozgursar):

 * keywords:  needs-testing changes-requested has-patch => changes-requested
     has-patch has-screenshots


Comment:

 == Patch Testing Report

 Patch Tested for Themes Twenty Ten, Twenty Eleven, Twenty Twelve:
 https://github.com/WordPress/wordpress-develop/pull/8236
 Patch Tested for Theme Twenty Thirteen: https://github.com/WordPress
 /wordpress-develop/pull/8324
 Patch Tested for Theme Twenty Fourteen: https://github.com/WordPress
 /wordpress-develop/pull/8325


 === Environment
 - WordPress: 7.0-alpha-61215-src
 - PHP: 8.2.29
 - Server: nginx/1.29.4
 - Database: mysqli (Server: 8.4.7 / Client: mysqlnd 8.2.29)
 - Browser: Chrome 144.0.0.0
 - OS: macOS
 - Theme: Twenty Twelve 4.7
 - MU Plugins: None activated
 - Plugins:
   * Code Snippets 3.9.4
   * Test Reports 1.2.1

 === Steps taken
 1. Activate one of the bundled themes Twenty Ten, Twenty Eleven, Twenty
 Twelve, Twenty Thirteen, Twenty Fourteen
 2. View Appearance > Customize > Site Identity
 3. Check Site Logo
 4. Confirm that Site Logo is not available.

 5. Apply patch PR8236 for Twenty Ten, Twenty Eleven, Twenty Twelve
 6. View Appearance > Customize > Site Identity
 7. ✅ Patch is solving the problem

 8. Apply patch PR8324 for Twenty Thirteen
 9. View Appearance > Customize > Site Identity
 10. ✅ Patch is solving the problem

 11. Apply patch PR8325 for Twenty Fourteen
 12. View Appearance > Customize > Site Identity
 13. ✅ Patch is solving the problem


 === Expected result
 - Each pull request successfully solves the issue and makes Site Logo
 available in the theme customize

 === Screenshots/Screencast with results
 ||= Twenty Ten Before =||= Twenty Ten After =||
 || [[Image(https://i.imgur.com/KDU6R3Z.png)]] ||
 [[Image(https://i.imgur.com/yMgmgvr.png)]] ||

 ----

 ||= Twenty Eleven Before =||= Twenty Eleven After =||
 || [[Image(https://i.imgur.com/PENGAJ0.png)]] ||
 [[Image(https://i.imgur.com/WR1OEjg.png)]] ||

 ----

 ||= Twenty Twelve Before =||= Twenty Twelve After =||
 || [[Image(https://i.imgur.com/EXDvRD5.png)]] ||
 [[Image(https://i.imgur.com/h5KuUhG.png)]] ||

 ----

 ||= Twenty Thirteen Before =||= Twenty Thirteen After =||
 || [[Image(https://i.imgur.com/Na7b6zX.png)]] ||
 [[Image(https://i.imgur.com/eKNtqRV.png)]] ||

 ----

 ||= Twenty Fourteen Before =||= Twenty Fourteen After =||
 || [[Image(https://i.imgur.com/FjjqDmH.png)]] ||
 [[Image(https://i.imgur.com/qJmDQQL.png)]] ||

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/61766#comment:18>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform