Skip to content

Fix form-submit styles by adding button classes to the submit-button in post-comments block#37245

Merged
aristath merged 2 commits intotrunkfrom
fix/comments-block-submit-classes
Dec 10, 2021
Merged

Fix form-submit styles by adding button classes to the submit-button in post-comments block#37245
aristath merged 2 commits intotrunkfrom
fix/comments-block-submit-classes

Conversation

@aristath
Copy link
Copy Markdown
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 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
Copy Markdown
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.

Comment thread packages/block-library/src/post-comments/index.php Outdated
Comment thread packages/block-library/src/post-comments/index.php Outdated
@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 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.

2 participants