Можно ли это как то упростить?

Question

[olya_097]

если да
то как именно
https://jsfiddle.net/fg1cz3sw/1/

Answer 1

[Одиночка Айс]

$('.main-header__button').each(function(){
  var btn = $(this);
  btn.off('click').on('click', function(e) {
    btn.parent().find('.main-header__input').trigger('focus');
    btn.parent().find('.main-header__search').addClass('main-header__search-active');
  });
});

Comments

[olya_097]

Одиночка Айс, спасибо
А можете немного обьяснить как работает? Зачем писать off

[Сергей delphinpro]

olya_097, так проще

$('.main-header__button').on('click', function(e) {
    var $parent = $(this).parent();
    $parent.find('.main-header__input').trigger('focus');
    $parent.find('.main-header__search').addClass('main-header__search-active');
});

[olya_097]

Сергей delphinpro, спасибо

[Одиночка Айс]

olya_097, off отключает повторное прокликивание по элементу, они имеют св-во накапливаться

Answer 2

[Пашенька]

Не совсем прямой ответ на ваш вопрос, скорее совет, как сэкономить память, сделать код расширяемым и оценить преимущества делегирования:

document.addEventListener('click', function (event) {

	switch (event.target.className) { // или
																    // var class = event.target.classList;
		case 'main-header__button':     // и в кейсах проверять class.contains('main-header__button')
			doSomething(event.target);
			break;

		case '...':
			...
			break;

		...

	}

});

function doSomething(target) {
	// выбрать в вашем случае относительно таргета, к примеру это сосед:
	target.nextElementSibling.focus();
	// выбрать в вашем случае относительно таргета, к примеру это сосед:
	target.previousElementSibling.classList.add('main-header__search-active');
}

И да, выбросьте в помойку джейквери, ему пора на покой.

Comments

[Одиночка Айс]

Типа давайте все дружно зубрить нативный js?

[Антон]

Он решает свою задачу. Ну и, давайте выкинем все ЯП, оставим ассемблер

[Пашенька]

Антон, джейквери давно уже не решает никаких задач

[Пашенька]

Одиночка Айс, не, типо давайте все дружно говнокодить. Я донес свой посыл, каждый сам пусть решит, как ему писать. Здесь человек учится, а учится первично говнокодить - не лучшая затея.

[Одиночка Айс]

like-a-boss, jQuery и говнокод... странные ассоциации. Наговнокодить я и на js могу, в виду слабых знаний