Automattic / newspack-theme

A theme for Newspack.
https://newspack.com
GNU General Public License v2.0
305 stars 66 forks source link

fix: correct subtitle order for WP 6.6 #2341

Closed laurelfulford closed 4 months ago

laurelfulford commented 4 months ago

All Submissions:

Changes proposed in this Pull Request:

This PR is a replacement for https://github.com/Automattic/newspack-theme/pull/2337, in hotfix form.

This PR updates where the article subtitle preview is inserted in the editor, to correct an order issue that happens with WP 6.6.

It also removes some unused code:

See 1207594452716169-as-1207669473649561

How to test the changes in this Pull Request:

  1. Start on a test site running the latest WP 6.6 RC.
  2. Create a post, and add a subtitle in the sidebar; note the editor order:

image

  1. Apply this PR and run npm run build.
  2. Confirm the editor order now looks better.

image

  1. Retest this PR on a site running WP 6.5.x, and confirm that it doesn't cause issues (if you're using the WordPress Beta plugin, you can also just roll back your original test site).

Other information:

matticbot commented 4 months ago

:tada: This PR is included in version 1.90.4 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket:

matticbot commented 4 months ago

:tada: This PR is included in version 1.91.0-alpha.2 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket:

matticbot commented 4 months ago

:tada: This PR is included in version 1.91.0-epic-ras-acc.3 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: