Since #2208585: Make #wrapper_attributes official and document it has now been committed, we need to update the documentation for the FAPI reference page, so document the #wrapper_attributes which has been in for a while in Drupal 8, but is only just now been made official.

Comments

googletorp’s picture

StatusFileSize
new2.27 KB

This is my first time writing a documentation patch, so not sure what's to be expected. I did some copy paste and it looks fine when reviewing the html file. I'm not sure if we need to add more documenation about #wrapper_attributes?

googletorp’s picture

Status: Active » Needs review
krina.addweb’s picture

Issue summary: View changes
jhodgdon’s picture

Status: Needs review » Closed (outdated)

This was a Drupal 8.x issue, and we do not have the HTML document for form elements in 8.x any more.

xumepadismal’s picture

Status: Closed (outdated) » Active

@jhodgdon what about FormElement topic? To me it looks as a right place to document this but I'm not sure.

Moreover, there are several undocumented properties for theme_form_element(): #label_attributes, #description_display and #wrapper_attributes.

Maybe we should document all of them in one patch/commit or?

jhodgdon’s picture

Status: Active » Closed (outdated)

Yes. Please file a new Drupal Core issue in component "documentation" to add the new documentation for missing properties to either FormElement or RenderElement as appropriate. This issue is about the Form API reference, which doesn't exist any more in Drupal 8.

(See also https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Render%21... which lists the docs for properties common to both form and render elements.)