It seems like a variation on the docs for 'every_page' option for drupal_add_js() and drupal_add_css() could be used, since it seems like it's similar.

CommentFileSizeAuthor
#2 952768.patch3.38 KBjhodgdon

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Issue tags: +Novice

Good catch - looks like a good project for a novice doc contributor...

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new 952768.patch3.38 KB

Here's a patch. (Novice issues are also good for tired doc contributors.) I did some minor cleanup of the doc of these two functions, as well as adding the missing @params.

duellj’s picture

duellj commented
Status: Needs review » Reviewed & tested by the community

Looks good, jhodgdon. I like the much more concise description of $every_page for drupal_add_library(). The description in drupal_add_js and drupal_add_css for $every_page could probably use some cleaning up as well; it took me a while to grok exactly what it does. This looks RTBC though.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented

Follow-up issue for drupal_add_css/js, as per comment #3 above:
#955776: 'every_page' parameter doc in drupal_add_js/css is confusing

Status: Fixed » Closed (fixed)
Issue tags: -Novice

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue tags: +Novice

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.