search

akiran / react-slick

React carousel component
http://react-slick.neostack.com/
MIT License
11.77k stars 2.11k forks source link

.slick-active not on correct slicks #1183

Open ipip2005 opened 6 years ago

ipip2005 commented 6 years ago

https://codesandbox.io/s/jlk8n3z9v9

  1. Inspect the elements, found slick-active class name is added properly to the first 4 slicks.
  2. Click > arrow button to flip to next page.
  3. Inspect again, now slick-active class is added to elements outside the viewport.

We can only detect the active items on the viewport when we can see accurately which ones are active slicks.

ipip2005 commented 6 years ago

Any updates on this one?

ipip2005 commented 6 years ago

Any updates?

maxval1 commented 6 years ago

The same issue demo https://codesandbox.io/s/ymn9kr8xjz

qqli1209 commented 5 years ago

The same issue demo https://codesandbox.io/s/xj4jwqm2zp

qqli1209 commented 5 years ago

As the aria-hidden goes with slick-active class, see here, the keyboard navigation is impacted.

qqli1209 commented 5 years ago

@akiran Would you please take a look at this issue? Thanks!

ravi-zengenti commented 5 years ago

Any update on this issue?