WordPress.org
Get WordPress

Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#21263 closed enhancement (fixed)

Twenty Twelve: <pre> styling

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

Description

I'd love to see the <pre> tag getting a little background colour love in Twenty Twelve.

Attachments (5)

pre-before.jpg​ (16.1 KB ) - added by sennza 14 years ago.
pre-after.jpg​ (43.6 KB ) - added by sennza 14 years ago.
21263.patch​ (514 bytes ) - added by sennza 14 years ago.
pre-after.1.png​ (75.1 KB ) - added by drewstrojny 14 years ago.
21263.1.patch​ (634 bytes ) - added by drewstrojny 14 years ago.

Download all attachments as: .zip

Change History (12)

@sennza
14 years ago

@sennza
14 years ago

@sennza
14 years ago

#1 follow-up: @helenyhou
14 years ago

  • Summary Twenty Twelve: I think the <pre> tag could use a background colour → Twenty Twelve: <pre> styling

That line height is out of control.

#2 @helenyhou
14 years ago

  • Milestone Awaiting Review → 3.5

#3 in reply to: ↑ 1 @sennza
14 years ago

Replying to helenyhou:

That line height is out of control.

Haha yeah I was gonna change that as well but thought I might cop some flack so I left it as is and just went for the bg colour change :)...I'm still trying to work all this core contribution stuff out...can you tell? ;)

#4 @obenland
14 years ago

  • Cc konstantin@… added

#5 @lancewillett
14 years ago

  • Cc lancewillett added

@drewstrojny
14 years ago

@drewstrojny
14 years ago

#6 @drewstrojny
14 years ago

Good idea sennza. <pre> styles now updated.

#7 @lancewillett
14 years ago

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

In [21340]:

Twenty Twelve: update pre element styles, props senzza and drewstrojny. Fixes #21263.

Note: See TracTickets for help on using tickets.