ansaurus

Question

Answer 1

+1 A:

Yes, delete the last four script tags and use this selector in the first one:

".hideshow-news, .hideshow-users, .hideshow-pages"

Cheers,

Boldewyn 2009-08-04 12:39:35

its not as simple as just changing the root selector, the click event will need a bit of reworking too, but this is basicly a good start yes

Sander 2009-08-04 12:54:05

Replace .click() with .toggle()

Soviut 2009-08-04 15:22:17

Answer 2

A:

A quick resolution to your problem follows. Delete all script tags and contents and just add this into your document preferably into the <head></head> section.

<script language="javascript">
    $(document).ready(function(){

      var arr = {"news", "pages", "users"};
      var x;
      for (x in arr){
        $(".hideshow-"+x).hide();

        $(".roll-li-"+x).click(function(){
          if ($(".hideshow-"+x).is(":hidden")) {
            $(".hideshow-"+x).slideDown("slow");
            $(".roll-li-"+x).removeClass('inactive');
          }else {
            $(".hideshow-"+x).slideUp("slow");
            $(".roll-li-"+x).addClass('inactive');
          }
        });
      }
    });
</script>

This should do what you need, i didn't test it though. If you have other places that you use this with same structure, change the array like this:

var arr = {"news", "pages", "users", "members", "others"};

but as Boldewyn says you should go over basics more.

Hope it helps, Sinan.

Sinan Y. 2009-08-04 15:10:22

Thanks!Great solution, and yes like you said, I should sit down and take one good deep look at jQuery!Thanks again.

AnzeT 2009-08-06 07:33:54

ansaurus

tags:

views:

answers:

Make jquery script global

related questions