Twenty Sixteen: Quote block Plain style has border in the editor

[nidhidhandhukiya]

Steps to reproduce the issue:-

1. Activate Twenty Sixteen theme.
2. Choose Quote block.
3. Add some text.
4. Now choose styles plain.

You can able to see no borders when plain is chosen in front side, but editor side there is no difference even when plain is used from styles.

[dilipbheda]

Test Report

Description

This report validates whether the indicated patch works as expected.

Patch tested: https://core.trac.wordpress.org/attachment/ticket/63424/63424.patch

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: Linux
• Theme: Twenty Sixteen 3.5
• MU Plugins: None activated
• Plugins:
  • Test Reports 1.2.0

Actual Results

1. ✅ Issue resolved with patch.

[sabernhardt]

GB64045 introduced :where() in the Quote block's theme.css, and then Twenty Sixteen's editor-blocks.css overrode the Plain style with default border styles.

Instead of adding the .editor-styles-wrapper class with :where(), one to raise specificity and one to lower it, I recommend

.wp-block-quote.is-style-plain {
	border: none;
}

[dilipbheda]

Updated patch file

[sainathpoojary]

Test Report

Description

This report validates whether the indicated patch works as expected.

Patch tested: https://core.trac.wordpress.org/attachment/ticket/63424/63424.2.patch

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 135.0.0.0
• OS: macOS
• Theme: Twenty Sixteen 3.5
• MU Plugins: None activated
• Plugins: Test Reports 1.2.0

Actual Results

✅ Issue resolved with patch.

Supplemental Artifacts

Before:
​https://ibb.co/kg78SmSp

After:
​https://ibb.co/tMHKDVjz

[sabernhardt]

I did not think about the possibility of using border-width: 0; earlier. If someone chooses the Plain style, border: none should be what they want. However, if Twenty Sixteen adds support for border options, border-width could allow adding a border even with the Plain style. (That decision could be part of the border options ticket instead.)

Anyway, 63424.2.patch works.

[dhruvang21]

Test Report

Description

This report validates whether the indicated patch works as expected.

Patch tested: https://core.trac.wordpress.org/attachment/ticket/63424/63424.2.patch

Environment

• WordPress: 6.9-alpha-60093-src
• PHP: 8.2.22
• Server: nginx/1.27.0
• Database: mysqli (Server: 8.0.39 / Client: mysqlnd 8.2.22)
• Browser: Chrome 136.0.0.0
• OS: Linux
• Theme: Twenty Ten 4.4
• MU Plugins: None activated
• Plugins:
  • Test Reports 1.2.0

Actual Results

1. ✅ Issue resolved with patch.

[harshgajipara]

Test Report

This report validates that the indicated patch addresses the issue.

Patch tested: https://core.trac.wordpress.org/attachment/ticket/63424/63424.patch

Environment

OS: Windows
PHP: 8.2.28
WordPress: 6.9-alpha-60093-src
Browser: Chrome
Theme: Twenty Sixteen
Plugins: None activated

Actual Results:

✅ Before patch: Has border in the editor
Backend: ​https://prnt.sc/bqsv0UId57bP
Frontend: ​https://prnt.sc/ojxJHOGoQRJC

✅ After patch: Border removed in the editor
Backend: ​https://prnt.sc/LuKwAZoIgbE-
Frontend: ​https://prnt.sc/A53ACFqRW_iI

[SirLouen]

Replying to harshgajipara:

This report validates that the indicated patch addresses the issue.

You have tested the wrong patch
Moreover, this report doesn't have needs-testing. No need for more tests.
If you are looking to find quality testing reports, come to Slack ​#core-test and we will advise on how to find.

[darshitrajyaguru97]

Test Report

Patch tested: https://core.trac.wordpress.org/attachment/ticket/63424/63424.2.patch

Environment:

OS: Windows
PHP: 8.2.28
WordPress: 6.9-alpha-60093-src
Browser: Chrome
Theme: Twenty Sixteen
Plugins: None activated

Actual Results:

✅ Before patch:
Backend: ​https://prnt.sc/aW5G2GPMw3my
Frontend: ​https://prnt.sc/ytvS0nG3NjM9

✅ After patch:
Backend: ​https://prnt.sc/yTostAEy7WNY
Frontend: ​https://prnt.sc/ZvOgFtfxEWN_

✅ Patch is working well as per desired solution

[karmatosed]

Assigning to myself to testing so I can then look to commit if that works because it has passed tests.

[karmatosed]

In 60342:

Twenty Sixteen: Makes sure quote block plain style is in editor.

The plain style for quote block was not showing in the editor despite showing on the front. This patch now makes sure it also does not show the border in editor.

Props nidhidhandhukiya, dilipbheda, sabernhardt, sainathpoojary, dhruvang21, harshgajipara, SirLouen, darshitrajyaguru97, jorbin.
Fixes #63424.

[sabernhardt]

Reopening for backport consideration.

[audrasjb]

In 60426:

Twenty Sixteen: Makes sure quote block plain style is in editor.

The plain style for quote block was not showing in the editor despite showing on the front. This patch now makes sure it also does not show the border in editor.

Reviewed by audrasjb, karmatosed.
Merges [60342] to the 6.8 branch.
Props nidhidhandhukiya, dilipbheda, sabernhardt, sainathpoojary, dhruvang21, harshgajipara, SirLouen, darshitrajyaguru97, jorbin.
Fixes #63424.