react-multi-carousel icon indicating copy to clipboard operation
react-multi-carousel copied to clipboard

keyBoardControl not working in version 2.6.3

Open pejman1 opened this issue 3 years ago • 10 comments

after updating to version 2.6.3 and 2.6.5 keyboard control does not work downgrade to the version 2.6.2 will solve the problem

windows 10 - react version 17

pejman1 avatar Aug 15 '21 10:08 pejman1

@YIZHUANG

pejman1 avatar Aug 23 '21 09:08 pejman1

@tegaadigu could https://github.com/YIZHUANG/react-multi-carousel/pull/252 have caused the issue?

YIZHUANG avatar Aug 23 '21 09:08 YIZHUANG

@YIZHUANG yes and i did create a fast follow fix for this:

Please review:

https://github.com/YIZHUANG/react-multi-carousel/pull/260

tegaadigu avatar Aug 23 '21 17:08 tegaadigu

@YIZHUANG yes and i did create a fast follow fix for this:

Please review:

#260

Thank you, is it possible to make it work for most of the users for now by removing the check for isChildOfCarousel? Cause I don't think I have the time to review the PR yet

YIZHUANG avatar Aug 23 '21 17:08 YIZHUANG

@YIZHUANG yes that logic has been removed in the above PR - it doesnt check fo childCarousel anymore.

https://github.com/YIZHUANG/react-multi-carousel/pull/260

tegaadigu avatar Aug 23 '21 22:08 tegaadigu

The problem has not fixed yet. I have just installed version 2.8.0 and keyBoardControl doesn't work for me but version 2.6.1 is okay.

pejman1 avatar Feb 26 '22 07:02 pejman1

The problem has not fixed yet. I have just installed version 2.8.0 and keyBoardControl doesn't work for me but version 2.6.1 is okay.

Can you open a PR for this?

YIZHUANG avatar Feb 26 '22 10:02 YIZHUANG

The problem has not fixed yet. I have just installed version 2.8.0 and keyBoardControl doesn't work for me but version 2.6.1 is okay.

Can you open a PR for this?

Hi @YIZHUANG , I see above that @tegaadigu already opened a PR to fix this 9 months ago. Is there some reason it's still not merged?

Sammii avatar Mar 01 '22 23:03 Sammii

@YIZHUANG i had a pr open to resolve this since may of 2021 - please merge when you can https://github.com/YIZHUANG/react-multi-carousel/pull/260

tegaadigu avatar Mar 01 '22 23:03 tegaadigu

@YIZHUANG still the problem has not fixed after a year. since version 2.6.1 !!!

pejman1 avatar Jun 19 '22 09:06 pejman1