Scribblex

ИМХО, название переменных и классов в первую очередь должны быть очевидными. Я – фан БЭМ-нотации. Но предпочитаю разделять классы следующим образом: для разметки / для поведения. Если надо повесить обработчик события, обычно поступаю так: class="some-bem-class js-handler"

Привет! transition не работает со свойством display, используйте opacity. Свойство transition должно быть определено сразу, а не вешаться / сниматься динамически
.slider-item {
opacity: 0;
vertical-align: middle;
transition: all 1000ms ease;
width: 100%;
}

.slider-item.active {
opacity: 1;
}