KevinBatdorf / liquidslider

A Responsive jQuery Content Slider
161 stars 62 forks source link

Added a fix for the greedy arrows class. #172

Closed tschallacka closed 5 years ago

tschallacka commented 5 years ago

According to https://github.com/KevinBatdorf/liquidslider/blob/master/src/js/jquery.liquid-slider.js#L173-L176 all liquidslider items are wrapped in a div with ls-wrapper. I added this as a prefix to the arrow class due to the fact that this greedy arrow selector messes up other parts of the site without a liquid slider By adding the prefix the arrows should stay within their designated area.

image

tschallacka commented 5 years ago

You're fast

KevinBatdorf commented 5 years ago

Was an easy fix. The code hasn't been changed much in 4 years though :D

Maybe I should do a rewrite