Opened 4 months ago
Last modified 3 months ago
#62010 new defect (bug)
Twenty Twelve: fix heading typography in the editor
Reported by: | dilip2615 | Owned by: | |
---|---|---|---|
Milestone: | Future Release | Priority: | normal |
Severity: | normal | Version: | |
Component: | Bundled Theme | Keywords: | has-patch has-screenshots good-first-bug |
Focuses: | css | Cc: |
Description
When activating the 'Twenty Twelve' theme and creating any post then display 'Add title' font size is very low compare to others
Attachments (5)
Change History (16)
#1
@
4 months ago
i'm new to core; i tried finding the file where the code for 'add new post' is written, but couldn't make it.
I want to contribute to this ticket, could anyone pl help me finding the related file?
@
4 months ago
adds selectors with .editor-styles-wrapper
and/or .wp-block-post-title
for typography rulesets
#2
@
4 months ago
- Component changed from General to Bundled Theme
- Keywords needs-design removed
- Version 6.6.1 deleted
The post title font-size
should remain as it is on the front: the rem
equivalent of 20px
for small screens, 22px
for screens wider than 600 pixels. It is very late to change that in a 12-year-old theme.
In the (iframe) editor, the size equals 21 pixels, which is a minor mismatch. However, the editor-blocks.css
file had defined styles to match the front with now-obsolete selectors such as .editor-post-title__block .editor-post-title__input
. The font-weight
is a more noticeable difference than the size.
The patch adds selectors that fit the current editor. Some of the proposed updates are not necessary within the iframe, but the non-framed version would use them. I also kept the original selectors for anyone with a WordPress 5.x site.
This ticket was mentioned in PR #7319 on WordPress/wordpress-develop by @iflairwebtechnologies.
4 months ago
#3
#4
@
4 months ago
@dilip2615
Generated new 62010.patch
Here you can see beforUpdated.png and afterUpdate.png
#5
@
4 months ago
- Milestone changed from Awaiting Review to Future Release
- Summary changed from Twenty Twelve: Add title size is very low. to Twenty Twelve: fix heading typography in the editor
I could have edited the ticket summary earlier. The theme's font size for post titles is 20px with narrower viewports and 22px for wider. 62010.diff may need additional iterations, but the patch should follow that direction.
#6
follow-up:
↓ 7
@
4 months ago
- Keywords needs-refresh good-first-bug added
Nice catch @dilip2615
As patch needs some iteration i'm going to add the needs-refresh
, also added good-first-bug
This ticket was mentioned in PR #7701 on WordPress/wordpress-develop by @anujrathore24.
3 months ago
#8
- Keywords needs-refresh removed
## PR Description
- Fix Heading Typography for Theme twenty twelve, for different viewports.
- Post titles are now set to 20px for narrow viewports and 22px for wider viewports, consistent with frontend design. Editor styles have been updated to reflect these sizes, providing a cohesive editing experience.
Trac ticket: https://core.trac.wordpress.org/ticket/62010
---
#9
@
3 months ago
Hello @sabernhardt,
Can you please review my PR and let me know if there are any changes required.
Thank You!
@dilip2615 commented on PR #7319:
3 months ago
#10
Please review it.
@Hemant Ahir commented on PR #7319:
3 months ago
#11
I've reviewed it, and everything is working fine on my end
Add title display very low, need to increase/improve font size.