Can this be shortened to be less code?

I want to know if I can break this to be less code.

I set slidetoggle to 4 different parts of the page, each div has its own id.

I hope it can be reduced to a few lines of code, since each div has the same slide speed as 200.

Here is the code:

$(document).ready(function() {
  $('#our-future-intro-slide').click(function() {
    //$(".our-future-intro").delay(2400).slideDown(3600);
    $(".our-future-intro").slideToggle(200);
  });
  $('#strategic-planning-click').click(function() {
    $("#strategic-planning").slideToggle(200);
  });
  $('#student-learning-click').click(function() {
    $("#student-learning").slideToggle(200);
  });
  $('#institutional-assessment-click').click(function() {
    $("#institutional-assessment").slideToggle(200);
  });
});
+3
source share
3 answers

You can put multiple identifiers in the selector, separated by commas. Then simply remove -clickor -slidefrom the id of the item that was clicked.

$(document).ready(function() {
  $('#strategic-planning-click,#student-learning-click,#institutional-assessment-click,#our-future-intro-slide').click(function() {
    var id = this.id.replace(/-click|-slide/, '');
    $("#" + id).slideToggle(200);
  });
});

EDIT:

@Ben Blank, .our-future-intro - , . , :

$(document).ready(function() {
  $('#strategic-planning-click,#student-learning-click,#institutional-assessment-click,#our-future-intro-slide').click(function() {
    var selector = this.id.replace(/^(.+)(-click|-slide)$/, function(str,p1,p2) { return (p2 == '-click') ? '#' + p1 : '.' + p1; })
    $(selector).slideToggle(200);
  });
});

EDIT:

:

, , HTML , .

$(document).ready(function() {
  $('#strategic-planning-click,#student-learning-click,#institutional-assessment-click,#our-future-intro-slide').click(function() {
    $(this).parent().next().slideToggle(200);
  });
});
+8

, , :

var objects = [ "our-future-intro", "strategic-planning", "student-learning", "institutional-assessment" ];

$.each(objects, function()
{
     $('#' + this + '-click').click(function() {
$("#" +  this).slideToggle(200);

})   });

0

HTML, . , HTML.

$("h1, h3").each(function(i, heading) {
    $('a', this).click(function() {
        heading.next('div').slideToggle(200);
    });
});

, (pastie):

<div>
    <h1><a href="..">one</a></h1>
    <div>content for one</div>

    <div>
        <h3><a href="..">two</a></h3>
        <div>content for two</div>
    </div>

    <div>
        <h3><a href="..">three</a></h3>
        <div>content for three</div>
    </div>

    ...
</div>
0

Source: https://habr.com/ru/post/1752681/


All Articles