Support for nested accordion (view in a view)

Thanks @Nor4a for the patch!

First a note that we follow drupal's coding standards on this project :)

On your patch:

• we use 2 spaces for tabulation
• if / else statements should start a new line
• and a few other things

Not a show stopper of course but it means the maintainer would have to spend more time to clean up your patch before committing which might get in the way of your patches getting accepted ;-)

About the patch itself, I will eventually find the time to play around with it. For now after a quick read, I can tell you that I'm not terribly a fan of using a static variable for this, and would like more people to test it and have a look at it in the mean time.

Thanks @Nor4a for following up. #5 looks a lot better.

I spent some time on this today. tested using views_field_view:

+++ b/views-accordion.js
@@ -45,6 +45,12 @@
+          ¶
+          //fix height of the accordeon if it was display:none prior to initialisation
+          $('.ui-accordion-header', $display).click(function(){ ¶
+            $(this).siblings('.ui-accordion-content').css('height','auto');
+            $(this).parents('.ui-accordion-content').css('height','auto');
+          });          ¶

This is not necessary. You can simply set heightStyle to content in the accordion settings, and the nested accordion's content will show up fine. We should probably document this so that people doing nested accordions are aware of it.

I have reworked your patch a bit so it applies to 8.1.x. It's just a one line change now.

I would like others to test this before committing it, specially people with running sites since we are changing one class that perhaps themers were relaying on.

I've had a thought about this, and we should keep the previous css class in place, and add the new one with the dom_id besides it, so not to break current installations that might have used this for theming already.

Attached patch does this, please kindly test and report what you find =)

If I nested a view inside of a view, inside of a 3rd view, this still doesn't seem to work.

Thanks for testing @heddn!
Was any of the three the same view / display? (wondering if they had the same dom_id)