Hi,

I needed to theme each rule link separately, so I thought that adding the rule link name as a class to the rule link wrapper span would do the trick.

What do you think? Jason.

CommentFileSizeAuthor
#4 rules_link-add-class-to-rule-link-1630998-4.patch768 bytesjasonawant
#1 rules_link-add-name-as-class-to-wrapper-1630998-1.patch947 bytesjasonawant
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jasonawant’s picture

jasonawant
he/him
Primary language English
Location New Orleans, USA
CreditAttribution: jasonawant commented
FileSize
rules_link-add-name-as-class-to-wrapper-1630998-1.patch947 bytes

Here is a patch file with this change. Please let me know what you think. Thanks, Jason.

mitchell’s picture

mitchell CreditAttribution: mitchell commented
Version: 7.x-1.0-beta3 » 7.x-1.x-dev
Status: Active » Needs review
sepgil’s picture

sepgil CreditAttribution: sepgil commented
Issue summary: View changes
Status: Needs review » Needs work

Seems usefull, but your diff is from your drupal site and not from rules link, and can therefore be not applied.

jasonawant’s picture

jasonawant
he/him
Primary language English
Location New Orleans, USA
CreditAttribution: jasonawant commented
Title: Add $rules_link->name as class to rule link wrapper span » Add $rules_link->name as class to rule link
Status: Needs work » Needs review
FileSize
rules_link-add-class-to-rule-link-1630998-4.patch768 bytes

Hi,

That was silly. How about this patch? Jason.

apmsooner’s picture

apmsooner CreditAttribution: apmsooner commented
Status: Needs review » Closed (works as designed)

Functionality already exists. Additional customization can be added to tpl.php file.