Make WordPress Core

Context Navigation

← Previous Ticket
Next Ticket →

Opened 13 years ago

Closed 13 years ago

#23580 closed defect (bug) (fixed)

Twenty Thirteen: better CSS selector for widget titles

Reported by:	aaroncampbell	Owned by:	lancewillett
Priority:	normal	Milestone:	3.6
Component:	Bundled Theme	Version:
Severity:	normal	Keywords:	has-patch commit
Cc:		Focuses:

Description

Some widgets wrap the title and content in an extra wrapper to allow additional styling. Since the style used for widget headers is .widget > h3 that breaks. Widget titles already have a widget-title class. Can we use that?

Attachments (2)

23580.diff (420 bytes ) - added by saracannon 13 years ago.: widget-title class
23580.2.diff (541 bytes ) - added by lancewillett 13 years ago.

Download all attachments as: .zip

Change History (6)

@saracannon
13 years ago

Attachment 23580.diff added

widget-title class

#1 @aaroncampbell
13 years ago

Keywords has-patch commit added; needs-patch removed

Thanks Sara. 23580.diff works for me.

@lancewillett
13 years ago

Attachment 23580.2.diff added

#2 @lancewillett
13 years ago

Probably doesn't even need the h3 in the selector. .widget .widget-title should work, and is simpler.

New patch also adjusts another case where it can use the class instead.

#3 @lancewillett
13 years ago

Summary Twenty Thirteen: Better CSS for Widget Title → Twenty Thirteen: better CSS selector for widget titles

#4 @lancewillett
13 years ago

Owner set to lancewillett
Resolution → fixed
Status new → closed

Twenty Thirteen: better selector for widget titles. Fixes #23580.

Note: See TracTickets for help on using tickets.

Download in other formats: