Twenty Fifteen: Button Letter Case is Not Working

[thakordarshil]

In Twenty Fifteen theme when we add button & try to add letter case in frontend & Backend (ex: Uppercase , Capitalize or lower case)

Steps to Reproduce the Issue :

• Activate Twenty Fifteen theme.
• Add Button in Any Page
• Add Letter Case in Button Block

Video Link: ​https://drive.google.com/file/d/1H3PLl5QFPFVdWCsU1v8UIwT70GjchiPB/view

[thakordarshil]

Patch Added

[pateljaymin]

Before Patch - Backend

[pateljaymin]

After Patch - Backend

[pateljaymin]

Before Patch - Frontend

[pateljaymin]

After Patch - Frontend

[pateljaymin]

Hi @thakordarshil

I have tested patch with wordpress 6.2.2 & Patch is working fine

[sabernhardt]

58526.diff could fix blocks where the user chose something other than uppercase, but it also changes every other Button block created with this theme.

The Letter Case control does not work in any theme that assigns a text-transform property to the .wp-block-button__link element, including:

• Twenty Fourteen
• Twenty Fifteen
• Twenty Sixteen
• Twenty Twenty

The ​default Button block styles probably could add the following to ensure support in any theme:

.wp-block-buttons[style*="text-transform"] .wp-block-button__link,
.wp-block-button[style*="text-transform"] .wp-block-button__link {
	text-transform: inherit;
}

Both selectors are necessary. The text-transform style is added to .wp-block-buttons when the Buttons block only has one Button block; multiple Button blocks in the same Buttons block get their own style attribute on the .wp-block-button container.

In case all themes that use uppercase should support the user control individually, I can add that to a patch for the bundled themes.

[sabernhardt]

supporting Letter Case control in Twenty Fourteen, Twenty Fifteen, Twenty Sixteen and Twenty Twenty

[panchalhimani711]

Before Patch-Backend

[panchalhimani711]

After Patch-Backend

[panchalhimani711]

Before Patch-Frontend

[panchalhimani711]

After Patch-Frontend

[panchalhimani711]

@sabernhardt

I have tested this patch: https://core.trac.wordpress.org/attachment/ticket/58526/58526.1.diff
for twentyfifteen theme & it's working fine.

Thanks.

[mukesh27]

Thank you, @thakordarshil, for the ticket.

Hey, @panchalhimani711, could you please review all the themes mentioned by @sabernhardt and provide a report?

The themes to check are:

• Twenty Fourteen
• Twenty Fifteen
• Twenty Sixteen
• Twenty Twenty

[panchalhimani711]

@mukesh27

Test Report for - https://core.trac.wordpress.org/attachment/ticket/58526/58526.1.diff Patch.

Theme - Twenty Fourteen

Backend :
=======
Before Patch Backend : ​https://prnt.sc/_BcvUA_2fdUa
After Patch Backend : ​https://prnt.sc/4l2ew1jAYOEF

Frontend :
=======
Before Patch Frontend : ​https://prnt.sc/sj_lyve3VBdi
After Patch Frontend : ​https://prnt.sc/caFy7_r0QmWC

Theme - Twenty Fifteen

Backend :
=======
Before Patch Backend: ​https://prnt.sc/qpSN3cpsdKCN
After Patch Backend : ​https://prnt.sc/B_erZQ6mMvY8

Frontend :
=======
Before Patch Frontend : ​https://prnt.sc/bvUBTE_zP2Rm
After Patch Frontend : ​https://prnt.sc/68EjnkLapsxl

Theme - Twenty Sixteen

Backend :
========
Before Patch Backend : ​https://prnt.sc/PJdafJDoVrzI
After Patch Backend : ​https://prnt.sc/mI8AdmjXI7mH

Frontend :
========
Before Patch Frontend : ​https://prnt.sc/BncoQgCapBHR
After Patch Frontend : ​https://prnt.sc/3wd_UZby68JM

Theme - Twenty Twenty

Backend :
=======
Before Patch Backend : ​https://prnt.sc/RCTYc8xKM3LE
After Patch Backend : ​https://prnt.sc/_jlZFP_M2b5p

Frontend :
========
Before Patch Frontend : ​https://prnt.sc/621EF2GHrL1Q
After Patch Frontend : ​https://prnt.sc/HoI2UxjhWpQ7

https://core.trac.wordpress.org/attachment/ticket/58526/58526.1.diff
Patch is Working fine in all mentioned theme

Thanks.

[mukesh27]

Thanks for the report @panchalhimani711

@sabernhardt already assign it in milestone. The patch solve issue for all the themes.

commit added keyword. Ping @audrasjb for review.

[audrasjb]

In 55997:

Twenty Fourteen: Fix Letter Case implementation.

This changeset fixes Letter Case control implementation on the Button block, on both front-end and in the Editor.

Props thakordarshil, pateljaymin, sabernhardt, panchalhimani711, mukesh27.
See #58526.

[audrasjb]

In 55998:

Twenty Fifteen: Fix Letter Case implementation.

This changeset fixes Letter Case control implementation on the Button block, on both front-end and in the Editor.

Props thakordarshil, pateljaymin, sabernhardt, panchalhimani711, mukesh27.
See #58526.

[audrasjb]

In 55999:

Twenty Sixteen: Fix Letter Case implementation.

This changeset fixes Letter Case control implementation on the Button block, on both front-end and in the Editor.

Props thakordarshil, pateljaymin, sabernhardt, panchalhimani711, mukesh27.
See #58526.

[audrasjb]

In 56000:

Twenty Twenty: Fix Letter Case implementation.

This changeset fixes Letter Case control implementation on the Button block, on both front-end and in the Editor.

Props thakordarshil, pateljaymin, sabernhardt, panchalhimani711, mukesh27.
See #58526.

[audrasjb]

Thanks everyone. I committed these fixes independently so they can be referenced one by one on theme changelogs (or even reverted if needed).

Closing as fixed.