Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#23580 closed defect (bug) (fixed)

Twenty Thirteen: better CSS selector for widget titles

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

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 11 years ago.
widget-title class
23580.2.diff (541 bytes) - added by lancewillett 11 years ago.

Download all attachments as: .zip

Change History (6)

@saracannon
11 years ago

widget-title class

#1 @aaroncampbell
11 years ago

  • Keywords has-patch commit added; needs-patch removed

Thanks Sara. 23580.diff works for me.

#2 @lancewillett
11 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
11 years ago

  • Summary changed from Twenty Thirteen: Better CSS for Widget Title to Twenty Thirteen: better CSS selector for widget titles

#4 @lancewillett
11 years ago

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

In 23490:

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

Note: See TracTickets for help on using tickets.