Make WordPress Core

Opened 8 years ago

Closed 6 years ago

#45865 closed enhancement (fixed)

Twenty Nineteen: Consider decreasing the font size for widget titles

Reported by: kjellr Owned by: ianbelanger
Priority: normal Milestone: 5.4.2
Component: Bundled Theme Version: 5.0
Severity: normal Keywords: has-patch commit early fixed-major
Cc: Focuses: css

Description

As per discussion in the Twenty Nineteen GitHub...

​https://github.com/WordPress/twentynineteen/issues/658

... we should consider decreasing the font size for widget titles. The current font size is the same size as post titles, which seems odd hierarchically.

Before:
https://cldup.com/G0B-GOk4I5-1200x1200.png

After:
https://cldup.com/G3HS80WOwW-2000x2000.png

---
This PR was ported over from the Twenty Nineteen GitHub here:
​https://github.com/WordPress/twentynineteen/pull/671

Attachments (2)

45865.patch​ (1.4 KB ) - added by kjellr 8 years ago.
45865.2.patch​ (1.4 KB ) - added by mukesh27 8 years ago.
Updated patch.

Download all attachments as: .zip

Change History (14)

@kjellr
8 years ago

This ticket was mentioned in ​Slack in #core-themes by anevins. ​View the logs.


8 years ago

@mukesh27
8 years ago

Updated patch.

#2 @mukesh27
8 years ago

@kjellr updated patch and remove h2 from style-rtl.css

#3 @kjellr
8 years ago

Thanks for catching that, @mukesh27!

#4 @ianbelanger
7 years ago

  • Focuses css added
  • Milestone Awaiting Review → 5.4
  • Owner set to ianbelanger
  • Priority lowest → normal
  • Status new → reviewing
  • Version 5.0.2 → 5.0

Reviewing for commit.

#5 @ianbelanger
7 years ago

  • Keywords commit early added
  • Milestone 5.4 → 5.5
  • Status reviewing → accepted

Patch still applies cleanly and the fix looks good. Marking commit and early and putting in the 5.5 milestone so we can get this enhancement in soon after 5.4 is released.

This ticket was mentioned in ​Slack in #core by david.baumwald. ​View the logs.


6 years ago

#7 @davidbaumwald
6 years ago

@ianbelanger Is this still on your list to include early in the 5.5 cycle? Is there anything anyone can assist with to move this forward?

#8 @ianbelanger
6 years ago

Yes it is @davidbaumwald, I'll get it committed this week.

#9 @ianbelanger
6 years ago

  • Resolution → fixed
  • Status accepted → closed

In 47586:

Bundled Themes: Twenty Nineteen decrease the font size for widget titles.

Update styles to decrease the font size for widget titles.

Props kjellr, mukesh27.
Fixes #45865.

#10 @ianbelanger
6 years ago

  • Keywords fixed-major added
  • Milestone 5.5 → 5.4.1
  • Resolution fixed
  • Status closed → reopened

Reopening for backport.

#11 @ianbelanger
6 years ago

  • Milestone 5.4.1 → 5.4.2

#12 @whyisjake
6 years ago

  • Resolution → fixed
  • Status reopened → closed

In 47831:

Bundled Themes: Twenty Nineteen decrease the font size for widget titles.
Update styles to decrease the font size for widget titles.

This commit brings the changes from [47586] to the 5.4 branch.

Props kjellr, mukesh27.
Fixes #45865.

Note: See TracTickets for help on using tickets.