I found that .js and .css files are not being added to each view element.

The code seams allright on expanding_formatter.module line 239.

I only could get it working when force add js and css with this:

drupal_add_css($module_path . '/css/expanding_formatter.css');
drupal_add_js($module_path . '/js/expanding_formatter.js');

after line 243 on the same file.

I tried to understand why it was not loading, but didn't came to a conclusion.

great module

CommentFileSizeAuthor
#10 css_and_js_not_being-2502947-10.patch1.37 KBsubhojit777
#4 expanding_formatter_resource_fix.patch1.58 KBcslevy
#1 expanding_formatter_resource_fix.patch1.42 KBcslevy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cslevy’s picture

cslevy CreditAttribution: cslevy at AG Prime commented
FileSize
expanding_formatter_resource_fix.patch1.42 KB

I had the same issue. Seems that you need to add the CSS/JS on every element. I created a patch for this.

cslevy’s picture

cslevy CreditAttribution: cslevy at AG Prime commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 1: expanding_formatter_resource_fix.patch, failed testing.

cslevy’s picture

cslevy CreditAttribution: cslevy at AG Prime commented
FileSize
expanding_formatter_resource_fix.patch1.58 KB

Re-created patch

cslevy’s picture

cslevy CreditAttribution: cslevy at AG Prime commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 4: expanding_formatter_resource_fix.patch, failed testing.

skadu’s picture

skadu
Primary language English
CreditAttribution: skadu commented

Using this formatter in a views display, the provided patch in #4 works.

Thanks cslevy.

3dnathaniel’s picture

3dnathaniel CreditAttribution: 3dnathaniel commented

Confirm patch in #4 works for me too. Thank you!

BYUStudies’s picture

BYUStudies CreditAttribution: BYUStudies commented

The patch in #4 works for me, too.

subhojit777’s picture

subhojit777
he/him
Primary language Bengali
CreditAttribution: subhojit777 as a volunteer and at Material commented
Status: Needs work » Needs review
FileSize
css_and_js_not_being-2502947-10.patch1.37 KB
subhojit777’s picture

subhojit777
he/him
Primary language Bengali
CreditAttribution: subhojit777 as a volunteer and at Material commented

This is just a reroll of #4

Status: Needs review » Needs work

The last submitted patch, 10: css_and_js_not_being-2502947-10.patch, failed testing.

subhojit777’s picture

subhojit777
he/him
Primary language Bengali
CreditAttribution: subhojit777 as a volunteer and at Material commented

There's no reason for failure of the test :(

teknojon’s picture

teknojon CreditAttribution: teknojon as a volunteer and commented

I manually applied the patch and it now works in views. It gave 'Hunk' fails when tried to apply with terminal window. I can provide the new expanding_formatter.module if anyone wants it?

teknojon’s picture

teknojon CreditAttribution: teknojon as a volunteer and commented
Issue tags: +It Works!