Make WordPress Core

Opened 5 weeks ago

Last modified 4 weeks ago

#62010 new defect (bug)

Twenty Twelve: fix heading typography in the editor

Reported by: dilip2615's profile dilip2615 Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-patch has-screenshots needs-refresh 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)

Screenshot_5.png (19.2 KB) - added by dilip2615 5 weeks ago.
Add title display very low, need to increase/improve font size.
62010.diff (1.9 KB) - added by sabernhardt 5 weeks ago.
adds selectors with .editor-styles-wrapper and/or .wp-block-post-title for typography rulesets
62010.patch (658 bytes) - added by iflairwebtechnologies 5 weeks ago.
beforUpdated.png (10.1 KB) - added by iflairwebtechnologies 5 weeks ago.
afterUpdate.png (10.7 KB) - added by iflairwebtechnologies 5 weeks ago.

Download all attachments as: .zip

Change History (12)

@dilip2615
5 weeks ago

Add title display very low, need to increase/improve font size.

#1 @navinrangar
5 weeks 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?

@sabernhardt
5 weeks ago

adds selectors with .editor-styles-wrapper and/or .wp-block-post-title for typography rulesets

#2 @sabernhardt
5 weeks 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.

#4 @iflairwebtechnologies
5 weeks ago

@dilip2615
Generated new 62010.patch
Here you can see beforUpdated.png and afterUpdate.png

#5 @sabernhardt
5 weeks 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.

Last edited 4 weeks ago by sabernhardt (previous) (diff)

#6 follow-up: @mukesh27
4 weeks 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

#7 in reply to: ↑ 6 @dilip2615
4 weeks ago

Replying to mukesh27:

Thank you for your support.

Note: See TracTickets for help on using tickets.