Javascript-форум (https://javascript.ru/forum/)
-   Элементы интерфейса (https://javascript.ru/forum/dom-window/)
-   -   Jquery, найти ближайший соседний элемент с классом (https://javascript.ru/forum/dom-window/65848-jquery-najjti-blizhajjshijj-sosednijj-ehlement-s-klassom.html)

ksa 11.11.2016 10:57

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

рони 11.11.2016 11:03

Цитата:

Сообщение от ksa
Мне понравился твой вариант,

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

Сообщение от ksa
//2 вариант

ok

рони 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 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 (Сообщение 434766)
Но чем провинились нексты у ТСа, я х/з... :D

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

Сообщение от ksa (Сообщение 434766)
P.S.
Это не верстка, а черти что! :-/

Самому тошно :blink:, но имеем то, что имеем.

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

рони 11.11.2016 12:15

Цитата:

Сообщение от eddin
Остановился на этих вариантах:

вам показали нормальные варианты ... зачем нужно делать
if ($moveout_body.is(":hidden"))

eddin 11.11.2016 13:49

Цитата:

Сообщение от рони (Сообщение 434784)
вам показали нормальные варианты ... зачем нужно делать
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;
	});
});

рони 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 11.11.2016 14:06

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

ksa 11.11.2016 14:07

Цитата:

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

Если что-то повторяется более одного раза - уже имеет. :yes:
Но цепочка методов вполне допустима. ;)
Т.е.

$(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 11.11.2016 15:09

Цитата:

Сообщение от ksa (Сообщение 434806)
Если что-то повторяется более одного раза - уже имеет. :yes:
Но цепочка методов вполне допустима. ;)
[/JS]

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


Часовой пояс GMT +3, время: 14:47.