WordPress.org

Make WordPress Core

Opened 2 years ago

Closed 23 months ago

Last modified 22 months ago

#24033 closed enhancement (fixed)

Bundled themes: add identifier to `style` elements added by the theme

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

Description

To assist in debugging CSS in a site's output, I suggest adding an ID attribute to identify style elements that default themes add to the head element.

Attachments (2)

24033.diff (2.7 KB) - added by lancewillett 2 years ago.
24033.2.diff (599 bytes) - added by SergeyBiryukov 23 months ago.

Download all attachments as: .zip

Change History (7)

@lancewillett2 years ago

comment:1 @lancewillett23 months ago

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

In 24001:

Bundled themes: add identifier to style elements added by the theme. Fixes #24033.

@SergeyBiryukov23 months ago

comment:2 follow-up: @SergeyBiryukov23 months ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

In Twenty Twelve, id attributes in twentytwelve_header_style() and twentytwelve_admin_header_style() are the same, however in Eleven and Thirteen they are different. Is that intentional?

comment:3 in reply to: ↑ 2 @lancewillett23 months ago

Replying to SergeyBiryukov:

In Twenty Twelve, id attributes in twentytwelve_header_style() and twentytwelve_admin_header_style() are the same, however in Eleven and Thirteen they are different. Is that intentional?

Not intentional, thanks for the catch, and patch.

comment:4 @lancewillett23 months ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In 24016:

Twenty Twelve: fix style element ID attribute value, props SergeyBiryukov. Closes #24033.

comment:5 @lancewillett22 months ago

In 24145:

Twenty Ten: fix style element ID attribute value, see #24033.

Note: See TracTickets for help on using tickets.