Форум

ksa · #11 (**permalink**) 11.11.2016, 10:57

Или

$(function(){
	$('.moveout_go').on('click', function () {
		//2 вариант    
		$(this).nextAll('.moveout_body').eq(0).slideToggle(200);
	});
});

рони · #12 (**permalink**) 11.11.2016, 11:03

Сообщение от ksa

Мне понравился твой вариант,

вариант 3 - ок!

Сообщение от ksa

//2 вариант

ok

рони · #13 (**permalink**) 11.11.2016, 11:04

<!DOCTYPE html>

<html>
<head>
  <title>Untitled</title>
  <meta charset="utf-8">
  <style type="text/css">
  </style>
  <script src="https://ajax.googleapis.com/ajax/libs/jquery/1.12.4/jquery.min.js"></script>

  <script>
$(function() {
$('.moveout_go').on('click', function () {
    //1 вариант $(this).nextAll('.moveout_body:first').slideToggle(200);
     $(' ~ .moveout_body:first', this).slideToggle(200); //2 вариант
    //3 вариант $('.moveout_body').eq($('.moveout_go').index(this)).slideToggle(200);
});
});
  </script>
</head>

<body>
<ul>
      <li><br>

      <span class="moveout_go">123</span><br>
      <div class="moveout_body">456</div>
      <br>


      <span class="moveout_go">123</span><br>
      <div class="moveout_body">456</div>
      <br>


      <span class="moveout_go">123</span><br>
      <div class="moveout_body">456</div>
      </li>
    </ul>


</body>
</html>

eddin · #14 (**permalink**) 11.11.2016, 11:21

Rise, ksa, рони, спасибо, господа!

рони, ваш сарказми и иронию оценил

Блин, ну нет у меня времени погружаться в jquery.((( Вы не представляете как бы я был рад, если бы у меня было время на изучение JS, но это только мечты...

Остановился на этих вариантах:

<script>
			$(function() {
				$(".moveout_go").on('click', function(){
					
					var $moveout_body = $(this).nextAll('.moveout_body').first();

					if ($moveout_body.is(":hidden")) {
						$moveout_body.slideDown(200);
					} else {
						$moveout_body.slideUp(200);
					}
					return false;
				});
			});
		</script>
		
		<script>
			$(function() {
				$(".moveout_go").on('click', function(){
					
					var moveout_body = $(this).nextAll('.moveout_body').eq(0);

					if ($moveout_body.is(":hidden")) {
						$moveout_body.slideDown(200);
					} else {
						$moveout_body.slideUp(200);
					}
					return false;
				});
			});
		</script>

не знаю какой выбрать, но мне кажется по производительности они на одном уровне, если не так, то помогите с выбором.

ps

Сообщение от ksa

Но чем провинились нексты у ТСа, я х/з...

Ну я писал выше, что может иногда BR быть больше одного.

Сообщение от ksa

P.S.
Это не верстка, а черти что!

Самому тошно

, но имеем то, что имеем.

pps Rise и рони что-то не даёт сайт карму поднять, пишет что сначала надо кому-то ещё это сделать. Странная система конечно.

рони · #15 (**permalink**) 11.11.2016, 12:15

Сообщение от eddin

Остановился на этих вариантах:

вам показали нормальные варианты ... зачем нужно делать

if ($moveout_body.is(":hidden"))

eddin · #16 (**permalink**) 11.11.2016, 13:49

Сообщение от рони

вам показали нормальные варианты ... зачем нужно делать

if ($moveout_body.is(":hidden"))

Блин, точно. Спасибо! Голова совсем не варит уже. Вот такой итоговый вариант получился, с выносом во внешний файл JS:

$(function() {
	$(".moveout_go").on('click', function(){
		var $this = $(this), $mb = $this.nextAll('.moveout_body').eq(0);
		$this.toggleClass("top_arr");
		$mb.slideToggle(200);
		return false;
	});
});

А ещё подскажите, плз, имеет ли смысл в данном случае объявлять переменные или лучше всё указать вот так?

$(function() {
	$(".moveout_go").on('click', function(){
		$(this).toggleClass("top_arr");
		$(this).nextAll('.moveout_body').eq(0).slideToggle(200);
		return false;
	});
});

рони · #17 (**permalink**) 11.11.2016, 14:00

Сообщение от eddin

имеет ли смысл в данном случае объявлять переменные

да

Сообщение от eddin

или лучше всё указать вот так?

тогда вот как можно

$(function() {
	$(".moveout_go").on('click', function(){
		$(this).toggleClass("top_arr")
		.nextAll('.moveout_body').eq(0).slideToggle(200);
		return false;
	});
});

eddin · #18 (**permalink**) 11.11.2016, 14:06

рони, спасибо!

ksa · #19 (**permalink**) 11.11.2016, 14:07

Сообщение от eddin

имеет ли смысл в данном случае объявлять переменные

Если что-то повторяется более одного раза - уже имеет.

Но цепочка методов вполне допустима.

Т.е.

$(function() {
	$(".moveout_go").on('click', function(){
		var obj=$(this);
		obj.toggleClass("top_arr");
		obj.nextAll('.moveout_body').eq(0).slideToggle(200);
		return false;
	});
});

Или

$(function() {
	$(".moveout_go").on('click', function(){
		$(this).toggleClass("top_arr").nextAll('.moveout_body').eq(0).slideToggle(200);
		return false;
	});
});

eddin · #20 (**permalink**) 11.11.2016, 15:09

Сообщение от ksa

Если что-то повторяется более одного раза - уже имеет.

Но цепочка методов вполне допустима.

[/JS]

Спасибо, запомню!

Форум

Справочник

Discord чат

Курсы javascript