Александр Захаров, Ну потому что с логикой неправильно тк вы наверняка не повесили сразу нужный класс + вы не удаляете класс. А так же смешиваете JS и jQuery
Примерно так получается.
max3wq, ну класс это такое себе решение, ну в крайнем случае я бы сделал вместоdata-id просто id т.е убрал бы ce из id и поменял в скрипте let id = self.attr('data-id'); на let id = self.attr('id');
Насколько я понимаю у вас нету колонок как таковых и у вас идут блоки друг за другом с неизвестной высотой... Masonry очень даже помоему вам подойдет... Чистого решения на флексах я не встречал... только или делать два столбика или masonry
как вариант сделать или два слайдера и объединить их скриптами или просто один слайдер и независимые блоки с отзывами которые тоже надо будет прикрепить к слайдеру при переключении слайда
Gorila Lopster, ну в моем по сути сделано, то что вам нужно) навешивание класса) и его удаление) А стрелочку уже можно в зависимости от .prjct.active делать как вам надо) Главное логику понять)
Люди... тут вопрос как бы воооообще о другом...
Короче говоря нету основных правил) Кому как удобно) Главное наверно это модульность, если у тебя можно вытащить блок и ничего не поедет, то наверно это правильно сверстанное!)
P.S
Ну там еще стили бы поправить чтобы блок не был таким здоровым с видео по ширине)
Лучше бы по умолчанию у вас была бы маска, а вот по клику на маску удалялся класс маски, а по клику куда то добавлялся... Просто как то логичнее было бы... ну хз...