Показать сообщение отдельно
  #1 (permalink)  
Старый 09.01.2019, 10:30
Аспирант
Отправить личное сообщение для phoenix200689 Посмотреть профиль Найти все сообщения от phoenix200689
 
Регистрация: 02.11.2016
Сообщений: 31

Возможно ли упростить/оптимизировать скрипт?
Приветствую!
Есть html-код и скрипт.
Со стилями сложнее, поэтому их нет (sorry).

<ul class="top-models__list-pros">
  <li class="pros">
    <div class="icon icon--pros"><span class="sign"></span></div>
    <div class="text" style="display: none;">Lorem ipsum dolor sit amet, consectetur adipiscing elit. Vivamus ullamcorper, velit vitae lacinia tristique, diam neque tincidunt sapien</div>
  </li>
  <li class="cons cons--active">
    <div class="icon icon--cons"><span class="sign"></span></div>
    <div class="text" style="display: none;">Lorem ipsum dolor sit amet, consectetur adipiscing elit. Vivamus ullamcorper, velit vitae lacinia tristique</div>
  </li>
  <li class="recomend">
    <div class="icon icon--recomend icon--active"><span class="sign"></span></div>
    <div class="text" style="display: inline-block;">Lorem ipsum dolor sit amet, consectetur adipiscing elit. Vivamus ullamcorper, velit vitae lacinia tristique, diam neque tincidunt sapien</div>
  </li>
</ul>


Скрипт работает, переключает вкладки.

Вопрос: возможно ли упростить скрипт, избавиться от вложенности?

$( document ).ready( () => {
    $('ul.top-models__list-pros > li').each(function() {
        $(this).find('div.icon').each(function() {
            $(this).on('click', function() {
                $(this).addClass('icon--active')
                    .closest('li')
                    .siblings()
                    .find('.icon')
                    .removeClass('icon--active');
            });
        });
    });
});


Заранее, спасибо за идеи!
Ответить с цитированием