Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
documentation
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
29 Dec 2013 at 15:48 UTC
Updated:
24 Jan 2014 at 16:40 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
jhodgdonGood idea... probably we should at least put @see references to PagerDefault and TableSort in this documentation. The 8.x docs need to be fixed up first -- there PagerDefault is turning into a link but not TableSort (missing the full namespaced class name). It still could use an @see.
Comment #2
stephaneqHere is a D8 patch
Comment #3
jhodgdonLooks good! The only thing is that @see lines should be at the bottom of the doc block, and they should be separated from the other documentation by a blank line.
Comment #4
jhodgdonActually... Given that those links apply to this paragraph only, it might make more sense to put them into the text of that paragraph directly rather than an @see (which will go into a See Also section at the bottom of the page, with no context). You can just put the URL in there and it will turn into a link on drupal.org.
Comment #5
stephaneqComment #6
jhodgdonPerfect, thanks!
Comment #7
jhodgdonThanks again! Committed to 8.x. We need something similar for 7.x, but obviously without the namespaces and with the correct 7.x class names.
Comment #8
stephaneqHere is the D7 patch
Comment #9
stephaneqOops, forgot status
Comment #10
jhodgdonThanks! I'll get this in.
Comment #11
jhodgdonThanks again! Committed to 7.x.