[wp-trac] [WordPress Trac] #24908: Proposed new filters for revisions
WordPress Trac
noreply at wordpress.org
Wed Oct 30 21:43:48 UTC 2013
#24908: Proposed new filters for revisions
------------------------------------+------------------------------
Reporter: adamsilverstein | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Revisions | Version:
Severity: normal | Resolution:
Keywords: has-patch dev-feedback |
------------------------------------+------------------------------
Comment (by DrewAPicture):
Inline docs notes on [attachment:24908.3.diff]:
Overall:
* There should be an empty line after the `@since` lines
* No leading slash on file paths in the duplicate comments
`show_revisions_split_view` filter:
* Use a docs-specific variable to represent the boolean in the parameter
line. Maybe `$bool` or something
* Specify the default at the end of the parameter description: "Default
true."
* Wrap the long description after "(two columns)"
`process_text_diff_html` filter:
* There are three parameters but only two are documented
* The long description is super cryptic. I had to read it a couple of
times to get it. Consider using as little jargon as possible. Also, the
various contexts need more explanation.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/24908#comment:7>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list