[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