Opened 11 years ago
Closed 11 years ago
#25608 closed defect (bug) (fixed)
Hook Docs (7): wp-admin/includes/class-wp-ms-themes-list-table.php
Reported by: | ShinichiN | Owned by: | kpdesign |
---|---|---|---|
Milestone: | 3.9 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Themes | Keywords: | has-patch |
Focuses: | docs | Cc: |
Description
Adding inline docs to the hooks in /wp-admin/includes/class-wp-ms-themes-list-table.php
Attachments (4)
Change History (15)
#2
@
11 years ago
- Keywords needs-patch added; has-patch removed
- Type changed from enhancement to defect (bug)
This still needs a new patch incorporating the changes suggested in comment:1.
#6
@
11 years ago
- Summary changed from Hook Docs: /wp-admin/includes/class-wp-ms-themes-list-table.php to Hook Docs (7): /wp-admin/includes/class-wp-ms-themes-list-table.php
25608.2.diff tightens up short descriptions and promotes consistency in individualizing hooks from each other.
#10
@
11 years ago
- Summary changed from Hook Docs (7): /wp-admin/includes/class-wp-ms-themes-list-table.php to Hook Docs (7): wp-admin/includes/class-wp-ms-themes-list-table.php
25608.3.diff expands on descriptions for a couple filters, and some minor changes for consistency. It also contains the documentation for both hooks in wp-admin/includes/class-wp-themes-list-table.php
, which are duplicates of hooks in wp-admin/includes/class-wp-ms-themes-list-table.php
.
I think this one's ready to go in.
Note: See
TracTickets for help on using
tickets.
Should be singular:
This sounds somehow … odd:
What does this mean?
For
theme_action_links_$stylesheet
the$stylesheet
part might deserve an explanation.In …
… and …
… the
</tr>
should be escaped or written without<
and>
, because unescaped HTML causes problems in the doc block parser.