Make WordPress Core

Opened 3 years ago

Last modified 7 months ago

#56638 reopened defect (bug)

Twenty Twenty-One: submit button colors inside a block with background color

Reported by: upadalavipul's profile upadalavipul Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 6.0.2
Component: Bundled Theme Keywords: has-patch changes-requested has-test-info
Focuses: css Cc:

Description

When we use Post Comments Form Block and change the block setting like background color & font color in the specific Twenty Twenty-One theme. then front side post comments button design breaks.

For better understanding, I have provided below video link.

URL: https://share.cleanshot.com/8SDCpoyOds4BwHDvtAFR

Attachments (2)

56638.patch (1.7 KB) - added by upadalavipul 3 years ago.
56638.1.diff (1.9 KB) - added by SirLouen 7 months ago.
Patch Refresh for 56638.patch

Download all attachments as: .zip

Change History (9)

@upadalavipul
3 years ago

#1 @upadalavipul
3 years ago

  • Keywords has-patch added; needs-patch removed
  • Resolution set to invalid
  • Status changed from new to closed

#2 @desrosj
3 years ago

  • Component changed from General to Editor
  • Keywords reporter-feedback added

Hi @upadalavipul,

I wanted to confirm that closing this ticket was intentional. If so, could you please give a brief summary as to why you've closed the report.

If it was not intentional, we should reopen so your problem can be confirmed and your patch can be tested.

#3 @upadalavipul
3 years ago

  • Resolution invalid deleted
  • Status changed from closed to reopened

#4 @upadalavipul
3 years ago

Hi @desrosj,

It was closed by mistake and now I have reopened it.

#5 follow-up: @sabernhardt
3 years ago

  • Component changed from Editor to Bundled Theme
  • Keywords reporter-feedback removed
  • Summary changed from front side post comments button design break in the post comments form block. to Twenty Twenty-One: submit button colors inside a block with background color

Using the Post Comments Form block with Twenty Twenty-One or any other 'classic' theme can create duplicate copies of the form, so I do not recommend that (GB45257).

However, there might be another situation where a submit button inside a block with a background color should not have the current styling.

#6 @karmatosed
18 months ago

  • Keywords needs-testing added
  • Milestone changed from Awaiting Review to Future Release

@SirLouen
7 months ago

Patch Refresh for 56638.patch

#7 in reply to: ↑ 5 @SirLouen
7 months ago

  • Keywords changes-requested has-test-info added; needs-testing removed

Test Report

Description

🟠 This report validates that the indicated patch works as expected, with some caveats (check Additional Notes)

Patch tested: https://core.trac.wordpress.org/attachment/ticket/56638/56638.1.diff

Environment

  • WordPress: 6.9-alpha-60093-src
  • PHP: 8.2.28
  • Server: nginx/1.27.5
  • Database: mysqli (Server: 8.4.5 / Client: mysqlnd 8.2.28)
  • Browser: Chrome 136.0.0.0
  • OS: Windows 10/11
  • Theme: Twenty Twenty-One 2.5
  • MU Plugins: None activated
  • Plugins:
    • Test Reports 1.2.0

Reproduction Steps

  1. Create a new post
  2. Add the "Comments Form" Block
  3. Change the Text Color and Background Color.

Actual Results

  1. 🟠 Issue resolved with a patch, but some code improvements could take place.

Additional Notes

  1. I have refreshed the patch because it was not applying (wrong patch format)
  1. Replying to sabernhardt:

    However, there might be another situation where a submit button inside a block with a background color should not have the current styling.

There is something that has been concerning me for a while. The use of global colour CSS variables in 2021 (this was never used before, from 2020 and before). Global Color CSS variables was released a year after 2021 was released and meant for Block Themes, as you say 2021 was the last classic editor, introducing this "feature" (that is a useless feature for a non-block theme since global colours are not editable in the interface, and they have brought a lot of havoc to 2021, most of the reports I've tested so far with this theme are related to the use of this)

  1. Conclusion: @upadalavipul, despite this patch works, I would suggest modifying to avoid using global colour CSS variables.

Supplemental Artifacts

🐞 Front End Pre-Patch

https://i.imgur.com/IDTFpfE.png

✅ Front End Post-Patch

https://i.imgur.com/HxZAnc3.png

Note: See TracTickets for help on using tickets.