Opened 13 months ago
Last modified 32 hours ago
#62773 assigned defect (bug)
Twenty Twenty: Pullquote Block Letter Case Setting Applies Only to Description in Editor, Not Cite
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | 7.0 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Bundled Theme | Keywords: | has-patch close 2nd-opinion |
| Focuses: | css | Cc: |
Description
The Pullquote block in the Twenty Twenty theme has an issue with the Letter Case setting:
In the editor, the Letter Case setting is only applied to the description (quote text) but not to the cite (citation/attribution).
On the frontend, the Letter Case setting works correctly and is applied to both the description and cite.
Attachments (3)
Change History (15)
#2
@
13 months ago
- Component changed from Bundled Theme to Themes
- Keywords has-patch added; needs-patch removed
#3
follow-up:
↓ 4
@
13 months ago
- Component changed from Themes to Bundled Theme
- Keywords needs-patch added; has-patch removed
@hiteshtalpada The patch 62773.patch seems wrong. isn't it?
#4
in reply to:
↑ 3
@
13 months ago
Replying to mukesh27:
@hiteshtalpada The patch 62773.patch seems wrong. isn't it?
@mukesh27, Can you please tell me what I'm doing wrong? I'm create diff using
git diffcommand.
Thanks
#5
@
12 months ago
Test Report
Description
This report validates whether the indicated patch works as expected.
Patch tested: https://core.trac.wordpress.org/attachment/ticket/62773/62773.patch
Environment
- WordPress: 6.8-alpha-59274-src
- PHP: 8.2.27
- Server: nginx/1.27.3
- Database: mysqli (Server: 8.0.40 / Client: mysqlnd 8.2.27)
- Browser: Chrome 131.0.0.0
- OS: macOS
- Theme: Twenty Twenty 2.8
- MU Plugins: None activated
- Plugins:
- Test Reports 1.2.0
Actual Results
- ✅ Issue resolved with patch. Pullquote block letter case is working fine in both in editor and frontend.
#6
@
12 months ago
@sukhendu2002 Thank you for validating the patch. Your efforts are greatly appreciated!
#7
@
12 months ago
Hi
@hiteshtalpada It looks like there is something wrong with the patch because it says that it is changing the separator block on line 890, but the separator is on line 630.
Of course, the separator should be untouched.
I am not sure what may have gone wrong: double check that you are creating the patch on the latest WordPress trunk.
#8
@
12 months ago
@poena, I agree with your observation that the CSS mentioned in the patch is not present in the latest trunk branch. Additionally, I was unable to reproduce this issue in the latest pull.
#9
@
12 months ago
Reproduction Report
Description
This report validates whether the issue can be reproduced.
Environment
- WordPress: 6.8-alpha-59274-src
- PHP: 8.2.22
- Server: nginx/1.27.0
- Database: mysqli (Server: 8.0.39 / Client: mysqlnd 8.2.22)
- Browser: Chrome 128.0.0.0
- OS: Linux
- Theme: Twenty Twenty 2.8
- MU Plugins: None activated
- Plugins:
- Test Reports 1.2.0
Actual Results
❌ I am not able to reproduce this issue.
Supplemental Artifacts
https://www.awesomescreenshot.com/video/35704758?key=b6093ae59a9ab3f236f246af9f1e1d5f
@
2 weeks ago
Pullquote blocks in the editor with patch: one block has default styling, the other has uppercase text
#10
@
2 weeks ago
- Keywords has-patch close 2nd-opinion added; needs-patch removed
- Milestone changed from Awaiting Review to 7.0
GB11610 started to deprecate the Pullquote block for WordPress 7.0, so it likely is not worth updating Twenty Twenty's editor styles for the block at this time. However, the patch is a small edit to the citation styles, and it worked for me with both default Pullquote styles and the uppercase setting. I'll set the milestone to make a decision whether to commit this patch or to close the ticket with no change at all.
Since WordPress 5.8 stopped adding block-library/theme.css in the editor, text-transform: none has been unnecessary, but setting it to inherit could work in older versions too.
Test Report
Description
This report validates that the indicated patch works as expected.
Patch tested: https://core.trac.wordpress.org/attachment/ticket/62773/62773.patch
Environment
- WordPress: 7.0-alpha-61215-src (r61418)
- PHP: 7.4.33
- Server: Apache/2.4.62 (Win64) PHP/7.4.33 mod_fcgid/2.3.10-dev
- Database: mysqli (Server: 9.1.0 / Client: mysqlnd 7.4.33)
- Browser: Firefox 146.0
- OS: Windows 11
- Theme: Twenty Twenty 3.0
- Plugins:
- Gutenberg 22.3.0
Actual Results
- ✅ Issue resolved with patch.
#11
@
11 days ago
Twenty Twenty: Pullquote Block Letter Case Setting Applies Only to Description in Editor, Not Cite
Description
The Pullquote block in the Twenty Twenty theme has an issue with the Letter Case setting:
In the editor, the Letter Case setting is only applied to the description (quote text) but not to the cite (citation/attribution).
On the frontend, the Letter Case setting works correctly and is applied to both the description and cite.
Patch tested: https://core.trac.wordpress.org/attachment/ticket/62773/62773.patch
Environment
- WordPress: 6.9
- PHP: 8.5.0
- Server: Apache/2.4.65 (Unix) mod_wsgi/5.0.2 Python/3.10 mod_fastcgi/mod_fastcgi-SNAP-0910052141 OpenSSL/1.1.1w
- Database: mysqli (Server: 5.7.44 / Client: mysqlnd 8.5.0)
- Browser: Chrome 143.0.0.0
- OS: macOS
- Theme: Twenty Twenty 3.0
- MU Plugins:
- Elementor Safe Mode 1.0.0
- Plugins:
- Test Reports 1.2.1
- WordPress Beta Tester 4.0.0
Actual Results
- ✅ Issue resolved with patch.
#12
@
32 hours ago
Test Repost for https://core.trac.wordpress.org/attachment/ticket/62773/62773.patch
Description:
==========
The Pullquote block in the Twenty Twenty theme had an inconsistency where the Letter Case setting applied only to the quote text in the editor, but not to the citation.
This caused a mismatch between the editor and frontend behavior. The issue has been resolved, and the Letter Case setting now applies consistently to both the quote and the citation in the editor.
Environment:
===========
PHP: 8.5.0
WordPress: 6.9
Browser: Chrome
Theme: Twenty Twenty
MU Plugin: Elementor Safe Mode 1.0.0
✅Patch is working very well.
The issue can be resolved by adding or updating the editor-specific styles in the theme to ensure the cite element inherits the Letter Case setting.