API page: http://api.drupal.org/api/drupal/includes--theme.inc/function/theme_links/7

Describe the problem you have found:

attributes: (optional) Attributes for the anchor, or for the tag used in its place if no 'href' is supplied.

This needs to say that if class is supplied, it must be an array:

'attributes' => array(
'class' => array('colorbox-load'),
),

this will crash:

'attributes' => array(
'class' => 'colorbox-load',
),

CommentFileSizeAuthor
#4 theme_inc-1314396-4.patch789 bytesdrupal_was_my_past
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhodgdon’s picture

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

Good idea, and probably a good project for a novice contributor.

joachim’s picture

joachim CreditAttribution: joachim commented

Indeed, and could be copied from the docs for http://api.drupal.org/api/drupal/includes--common.inc/function/l/7.

drupal_was_my_past’s picture

drupal_was_my_past CreditAttribution: drupal_was_my_past commented
Assigned: Unassigned » drupal_was_my_past

Working on this at BADCamp.

drupal_was_my_past’s picture

drupal_was_my_past CreditAttribution: drupal_was_my_past commented
Status: Active » Needs review
FileSize
theme_inc-1314396-4.patch789 bytes

Patch attached.

joachim’s picture

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

Looks good! :)

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented

Indeed, thanks!

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Version: 8.x-dev » 7.x-dev

Thanks! Committed and pushed to 8.x, moving to 7.x.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 7.x. Thanks!

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