Сделал слайдер, а кнопки не работают?

Question

[Игорь Касперский]

Все сделал, но кнопки не работают, код есть, взгляните что там
https://jsfiddle.net/o0yx0e2n/

Answer 1

[Алексей Уколов]

Uncaught ReferenceError: next is not defined @ (index):117
Uncaught ReferenceError: prev is not defined @ (index):116

Взглянул.

Answer 2

[Serdji]

Может быть стоило бы подвесить событие не в самом теги span а в JS ? например так.

<span class="prev">&lt;</span>
		    <span class="next">&gt;</span>

$(  '.prev'  ).on('click',  function() {
	var newSlider = sliderInt - 1;
	showSlider(newSlider);
});

$( '.next' ).on('click',  function() {
	var newSlider = sliderInt + 1;
	showSlider(newSlider);
});

Если все равно работать не будет, можно попробовать сделать дилигирование

$(  docunent  ).on('click',  '.prev',  function() {
	var newSlider = sliderInt - 1;
	showSlider(newSlider);
});

$(  documetn  ).on('click',  '.next',  function() {
	var newSlider = sliderInt + 1;
	showSlider(newSlider);
});

Answer 3

[Pavel Torbeev]

у Serdji почти верное решение. Еще необходимо вынести переменную count из функции
Решение:
https://jsfiddle.net/o0yx0e2n/3/

Comments

[Игорь Касперский]

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

[Pavel Torbeev]

Возможно это происходит из-за автоплея. В момент переключения срабатывает автоплей и т.к. две картинки, слайдер снова переключается на ту же

Answer 4

[Александр Зачиналов]

попробуйте заводить функции иначе

var prevSlideW = function(){
	var newSlider = sliderInt - 1;
    showSlider(newSlider);
};

Answer 5

[Виталий Инчин ☢]

Либо вынеси все функции в глобальную область видимости, либо объявляй их иначе:

window.prev = function(){
  ...
}

window.next = function(){
  ...
}

А лучше повесь нормальные обработчики, это же jQuery, черт подери!