WordPress.org
Get WordPress

Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#21262 closed enhancement (fixed)

Twenty Twelve: I think the <address> tag needs a little more line-height

Reported by: sennza Owned by: lancewillett
Priority: normal Milestone: 3.5
Component: Bundled Theme Version:
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

It's only a little tweak but I think it makes a difference!

Attachments (4)

address-before.jpg​ (24.9 KB ) - added by sennza 14 years ago.
address-after.jpg​ (25.5 KB ) - added by sennza 14 years ago.
21262.patch​ (526 bytes ) - added by sennza 14 years ago.
21262.1.patch​ (417 bytes ) - added by drewstrojny 14 years ago.

Download all attachments as: .zip

Change History (9)

@sennza
14 years ago

@sennza
14 years ago

@sennza
14 years ago

#1 @SergeyBiryukov
14 years ago

  • Milestone Awaiting Review → 3.5

#2 @obenland
14 years ago

  • Cc konstantin@… added

@drewstrojny
14 years ago

#3 @drewstrojny
14 years ago

Good catch sennza. I tweaked your styling a bit to use the default line height in the updated patch. Keep in mind for future patches, we're using unitless line-height values:

​http://meyerweb.com/eric/thoughts/2006/02/08/unitless-line-heights/

#4 @lancewillett
14 years ago

  • Cc lancewillett added

#5 @lancewillett
14 years ago

  • Owner set to lancewillett
  • Resolution → fixed
  • Status new → closed

In [21338]:

Twenty Twelve: give address element a little more line-height, props sennza and drewstrojny. Fixes #21262.

Note: See TracTickets for help on using tickets.