Opened 9 months ago
Closed 7 months ago
#21675 closed enhancement (fixed)
Twenty Twelve: review and clarify code comments
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.5 |
| Component: | Bundled Theme | Version: | |
| Severity: | trivial | Keywords: | has-patch |
| Cc: | xoodrew@… |
Description
Some functions are mis documented and others could be a bit more verbose
Attachments (7)
Change History (13)
comment:1
lancewillett — 9 months ago
- Owner set to lancewillett
- Status changed from new to accepted
- Summary changed from The documentation in twentytwelve needs a bit of clarification to Twenty Twelve: review and clarify code comments
comment:2
lancewillett — 9 months ago
- Priority changed from low to normal
comment:3
lancewillett — 9 months ago
In [21617]:
comment:4
lancewillett — 9 months ago
- Resolution set to fixed
- Status changed from accepted to closed
In [21618]:
DrewAPicture — 7 months ago
DrewAPicture — 7 months ago
DrewAPicture — 7 months ago
DrewAPicture — 7 months ago
DrewAPicture — 7 months ago
comment:5
DrewAPicture — 7 months ago
- Cc xoodrew@… added
- Resolution fixed deleted
- Status changed from closed to reopened
I've attached some cleanup diffs for consideration per @lancewillett's go-ahead. A great deal of this is spacing out blocks of markup and code so it's more readable and easier to follow. Mixed in are several missing end-of-element HTML comments and a couple of typo/grammar fixes. I've grouped the changes into multiple diffs so it's easier to take or leave some but not all.
comment:6
DrewAPicture — 7 months ago
- Resolution set to fixed
- Status changed from reopened to closed
Re-closing as fixed.
Note: See
TracTickets for help on using
tickets.

Thanks—will review and get the clarifications in. Except maybe the anteater comment. :)