WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#29961 closed defect (bug) (fixed)

Twenty Fifteen: Inline docs for header.php (Site-Branding class)

Reported by: NikV Owned by: iandstewart
Milestone: 4.1 Priority: normal
Severity: normal Version: 4.1
Component: Bundled Theme Keywords: has-patch
Focuses: docs Cc:

Description

Making my initial commit and documenting the site-branding class in the header.php in Twenty Fifteen

Attachments (2)

header.php.diff (585 bytes) - added by NikV 7 years ago.
header_1.php.diff (681 bytes) - added by NikV 7 years ago.

Download all attachments as: .zip

Change History (9)

@NikV
7 years ago

#1 @obenland
7 years ago

  • Component changed from Themes to Bundled Theme

#2 follow-up: @iamtakashi
7 years ago

@NikV, please make your diff from the root.

@NikV
7 years ago

#3 in reply to: ↑ 2 @NikV
7 years ago

Replying to iamtakashi:

@NikV, please make your diff from the root.

Alright, updated file from root.

#4 @iandstewart
7 years ago

  • Keywords has-patch added

#5 @iandstewart
7 years ago

  • Milestone changed from Awaiting Review to 4.1

#6 @DrewAPicture
7 years ago

  • Focuses docs added

#7 @iandstewart
7 years ago

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

In 29928:

Twenty Fifteen: document the site-branding class in header.php.

Props NikV, fixes #29961.

Note: See TracTickets for help on using tickets.