Неплохо. Хорошо, что без jquery Сделай какой-нибудь hover(() => {autoplay =

Question

Неплохо. Хорошо, что без jquery Сделай какой-нибудь hover(() => {autoplay =

false;})
Те картинки слайдера, которые меньше контейнера, надо растягивать. При интеграции это будет проблема
Стрелки чем-то выделить бы. Например filter: drop-shadow(0px 0px 5px #fff); может что-то в таком стиле. На темном фоне ее не видно
Хреново, что самый верхний контейнер с overflow:hidden, из-за этого стрелки не вынести наружу. А так бы опцией указывать
> if(!switches) {
controllersWrap.style.display = 'none';
А что это? Не могу найти controllersWrap в DOM. Судя по стилю это навигация снизу. Если в слайдре будет много элементов, в этот контейнер будет помещено больше превьюшек и они просто обрежутся. Нужно как-то перехватить ширину итоговую и если она больше слайдера, то навигацию добавить. Это не сложно. Еще как-то выделять в превьюшках надо активный элемент. Приписывать ему класс active какой-нибудь и типо такого получилось чтобы: http://skrinshoter.ru/s/301216/5xp8bs.png
Не нравится, что ты оперируешь так стилями. Лучше б, таки, css подтянуть и прописывать классы, будет гибко. Сейчас хз как кастюмизировать под себя
Еще фича - сделать атрибут title="Text" и это выводить на слайдере. Но контейнер, в котором будет этот текст - чтобы можно было кастюмизировать своим стилем. (например на первом слайдере это будет желтый прямоугольник с одним текстом слева-снизу, а на втором - справа-снизу)

#frontend #html #javascript #programming #russian

0

30.12.2016

1 ответов

6 просмотров

Umid Musaev · Accepted Answer

Спасибо большое. То что я ожидал. Есть пару моментов которые я недопонял: 1) "Не нравится, что ты оперируешь так стилями. Лучше б, таки, css подтянуть и прописывать классы, будет гибко. Сейчас хз как кастюмизировать под себя" Имеете ввиду к main.js и файл со стилями поставлять? Изначально я расчитывал на слайдер в 1 файле, поэтому уже настроился на 1 файл only. 2)"Еще фича - сделать атрибут title="Text" и это выводить на слайдере. Но контейнер, в котором будет этот текст - чтобы можно было кастюмизировать своим стилем. (например на первом слайдере это будет желтый прямоугольник с одним текстом слева-снизу, а на втором - справа-снизу)" Здесь я тоже не особо смог понять, но может сделать поддерку блоков? Т.е. чтоб не только картинки можно было передавать слайдеру, но и блоки, а блоки уже самому настраивать и пихать в него что захочешь? Было бы гибче. На счёт всего остально, всё учту и буду дорабатывать. Так же на счёт коду хочу вас спросить. Один человек сказал мне, что стоит всё переписать. Мол настолько всё плохо. Но чем именно он так и не ответил. Что вы думаете на счёт этого?

169 похожих чатов

Неплохо. Хорошо, что без jquery Сделай какой-нибудь hover(() => {autoplay =

1 ответов

Похожие вопросы