Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#38767 closed enhancement (fixed)

Twenty Seventeen: Table of contents numbers

Reported by: samikeijonen's profile sami.keijonen Owned by: davidakennedy's profile davidakennedy
Milestone: 4.7 Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-patch
Focuses: Cc:

Description

Should table of contents have numbering before the release?

Attachments (1)

38767.diff (6.8 KB) - added by karmatosed 7 years ago.

Download all attachments as: .zip

Change History (5)

@karmatosed
7 years ago

#1 @karmatosed
7 years ago

  • Keywords has-patch added

I've added a patch with the changes to table of contents. @davidakennedy if you can take a look please. Thanks @sami.keijonen for mentioning we should do this.

#2 @davidakennedy
7 years ago

  • Milestone changed from Awaiting Review to 4.7

#3 @helen
7 years ago

This seems fairly low priority to me, so take that into account as I throw this question out there - what is the benefit of numbering things? Seems like a maintenance burden and searching for numbers alone isn't really all that great of an index in a CSS file, so you end up typing some amount of the title string anyway.

#4 @davidakennedy
7 years ago

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

In 39216:

Twenty Seventeen: Adds numbers to the table of contents in style.css

Props karmatosed.

Fixes #38767.

Note: See TracTickets for help on using tickets.