Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#21263 closed enhancement (fixed)

Twenty Twelve: <pre> styling

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

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 12 years ago.
pre-after.jpg (43.6 KB) - added by sennza 12 years ago.
21263.patch (514 bytes) - added by sennza 12 years ago.
pre-after.1.png (75.1 KB) - added by drewstrojny 12 years ago.
21263.1.patch (634 bytes) - added by drewstrojny 12 years ago.

Download all attachments as: .zip

Change History (12)

@sennza
12 years ago

@sennza
12 years ago

@sennza
12 years ago

#1 follow-up: @helenyhou
12 years ago

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

That line height is out of control.

#2 @helenyhou
12 years ago

  • Milestone changed from Awaiting Review to 3.5

#3 in reply to: ↑ 1 @sennza
12 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
12 years ago

  • Cc konstantin@… added

#5 @lancewillett
12 years ago

  • Cc lancewillett added

#6 @drewstrojny
12 years ago

Good idea sennza. <pre> styles now updated.

#7 @lancewillett
12 years ago

  • Owner set to lancewillett
  • Resolution set to fixed
  • Status changed from new to closed

In [21340]:

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

Note: See TracTickets for help on using tickets.