The call to drupal_get_path() in _responsive_tables_filter_add_js_css() should be next to where it is needed.

CommentFileSizeAuthor
#2 responsive_tables_filter-refactor-2657618-2-D7.patch1.51 KBLiam Morland
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Liam Morland created an issue. See original summary.

Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland at University of Waterloo commented
Assigned: Liam Morland » Unassigned
Status: Active » Needs review
FileSize
responsive_tables_filter-refactor-2657618-2-D7.patch1.51 KB

Status: Needs review » Needs work

The last submitted patch, 2: responsive_tables_filter-refactor-2657618-2-D7.patch, failed testing.

Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland at University of Waterloo commented
Status: Needs work » Needs review

mark_fullmer’s picture

mark_fullmer
he/him
Primary language English
Location Tucson
CreditAttribution: mark_fullmer as a volunteer commented
Status: Needs review » Fixed

Every little bit of performance helps, eh? Passes behat tests. Committed to 7.x-1.x-dev.

Thanks, Liam!

Status: Fixed » Closed (fixed)

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