Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
http://api.drupal.org/api/function/theme_fieldset/7
It looks to me like it also uses the "id" property.
Comment | File | Size | Author |
---|---|---|---|
#8 | 523034b.patch | 890 bytes | jhodgdon |
#2 | 523034.patch | 779 bytes | jhodgdon |
Comments
Comment #1
jhodgdonPlease file API doc issues under Project "Drupal", component "documentation". Project "Documentation" is only for handbook docs on drupal.org, not for API issues displayed on api.drupal.org (which are in the code). Thanks!
That aside, another issue with the documentation is that the components should all have # before them: #attributes, #id, etc.
Comment #2
jhodgdonHere is a patch. It brings the function header doc into compliance with current Doxygen standards (3rd person summary, 80 character lines), adds # before the attributes, and adds the #id attribute to the list.
Comment #3
jhodgdonComment #4
NancyDru-- it can nicely be used anywhere a collapsible fieldset is desired - not just in forms.
Comment #5
jhodgdonTrue that a fieldset can be used for other things, though it is part of the Form API, so it's probably mostly used in forms and similar pages.
So do you have a suggestion of a better one-line description (80 characters or less) of the function besides the existing "Format(s) a group of form items"? I had just left it as it was (aside from changing the verb tense). I can't think of another concise way of describing what a fieldset is, off-hand.
Comment #6
NancyDruHow about, "Formats a set of information, such as a group of form items."
Why does it have to be one line?
Comment #7
jhodgdonThe first line of any doc header needs to be a one-line, 80-character summary of what the function does. It is supposed to be short, because it is used in lists of functions, such as those on this page
http://api.drupal.org/api/file/includes/form.inc/7
So we have made the standard that it should be one 80-character line.
Comment #8
jhodgdonHow about this patch?
Comment #9
NancyDruworks for me
Comment #10
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD.