Skip to content

Conversation

@aristath
Copy link
Member

@aristath aristath commented Dec 9, 2021

Description

Fixes the issue reported in WordPress/twentytwentytwo#281
Should be backported to WP5.9.
Since we split the post-comments block this doesn't work as expected because these changes were moved to the post-comments-form block (which is not being backported to WP Core yet). The changes must therefore also exist in the post-comments block for backwards-compatibility purposes with WP 5.9.

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • I've tested my changes with keyboard and screen readers.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.
  • I've updated all React Native files affected by any refactorings/renamings in this PR (please manually search all *.native.js files for terms that need renaming or removal).

@aristath aristath added the Backport to WP 6.9 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Dec 9, 2021
@aristath aristath requested a review from ajitbohra as a code owner December 9, 2021 07:14
@youknowriad youknowriad requested a review from gziolo December 9, 2021 08:50
@noisysocks noisysocks added [Block] Comments (legacy) The legacy mode of the Comments block (formerly known as Post Comments) [Type] Bug An existing feature does not function as intended labels Dec 10, 2021
Copy link
Member

@noisysocks noisysocks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested this locally and it fixed the bug identified in WordPress/twentytwentytwo#281 👍

Two comments, but I'll pre-approve so that you can merge on Friday in time for beta 3.

@aristath aristath force-pushed the fix/comments-block-submit-classes branch from bd5a9d8 to cf4200e Compare December 10, 2021 08:44
@aristath aristath merged commit 26fff37 into trunk Dec 10, 2021
@aristath aristath deleted the fix/comments-block-submit-classes branch December 10, 2021 12:59
@github-actions github-actions bot added this to the Gutenberg 12.2 milestone Dec 10, 2021
@noisysocks noisysocks removed the Backport to WP 6.9 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Dec 13, 2021
noisysocks pushed a commit that referenced this pull request Dec 13, 2021
…in post-comments block (#37245)

* Add button classes to the comments form

* remove function_exists check
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Block] Comments (legacy) The legacy mode of the Comments block (formerly known as Post Comments) [Type] Bug An existing feature does not function as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants