Нормально ли сверстан слайдер?

Question

[gleendo]

Пытаюсь изучать JS, дошел уже до клиентской части, попробовал сверстать слайдер, вроде как что то да получилось.
Вот демка слайдера - codepen.io/evg_/pen/xOdvpE?editors=0010

Делал как знал, поэтому хочется узнать на правильном ли пути я, или я что то не правильно сделал, поэтому,
посмотрите пожалуйста на реализацию самого слайдера и верстку, какие есть ошибки, что можно переделать, чтобы было лучше? Что сделано не правильно?

Answer 1

[xmoonlight]

1. фон слайдера "скачет", не корректно он работает: то текущий, то следующий то через два вперёд отображает... (что-то там с порядковыми формулами...)
2. кол-во слайдеров надо брать из тэгов, а не через константу задавать
3. при уменьшении (responsitive) основная картинка слайдера не умещается в окне слайдера.
Сходу, вроде всё.

Comments

[gleendo]

xmoonlight На счет фона, пока что убрал перекрутку. Количество слайдов указал через длину массива слайдов.
А на счет фона слайда, как сделать так чтобы умещалась картинка? если она начнет сжиматься то это кажется не очень будет, сильное искажение будет.

За ответ, спасибо

[xmoonlight]

evg_: может vw:100% и height:100% ? что-то в этом духе надо мудрить....

Answer 2

[Фёдор Ананин]

Что-то явно пошло не так, после того, как прокрутка началась по второму кругу умерло переключение фоновой картинки.

И, как по мне, было бы хорошим тоном было бы прокрутку задавать не в пикселях, а как-то иначе... в штуках, например :) И количество слайдов по идее должно высчитываться само, а не задаваться вручную.