Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Починить слайдер #65

Open
qqpayne opened this issue Aug 27, 2021 · 1 comment
Open

Починить слайдер #65

qqpayne opened this issue Aug 27, 2021 · 1 comment
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@qqpayne
Copy link
Member

qqpayne commented Aug 27, 2021

С добавлением возможности выбирать дату кликом в слайдер-календарь закрался и баг.

По какой-то причине слайдер перехватывает клики и не дает записаться в очередь. Нужно пофиксить.

Выпущен хотфикс #64

@qqpayne qqpayne added the bug Something isn't working label Aug 27, 2021
@qqpayne qqpayne linked a pull request Aug 27, 2021 that will close this issue
@qqpayne qqpayne added the help wanted Extra attention is needed label Aug 28, 2021
@qqpayne
Copy link
Member Author

qqpayne commented Aug 28, 2021

Как воспроизвести баг:

  1. Открываете на телефоне или в браузере с симуляцией тач-эвентов карусель
  2. Нажимаете на любой тайл
  3. Нажимаете в любом месте экрана (через произвольное время)
  4. Тайл вернется на прошлый

Судя по всему, причина бага в том, что карусель при нажатии на тайл воспринимает это как начало свайпа, но считает его незавершенным. Нажатие в любое местом экрана завершает свайп, карусель считает что свайп был нулевым и возвращается на место предыдущего тайла.

Пробовал ставить на кнопку onTouchStart и прочие хэндлеры и останавливать эвенты. Не помогало. Карусель сама обрабатывает всю область нахождения тайлов.

Казалось бы, при нажатии на тайл эмитится также и onTouchEnd и он должен завершать свайп. Но почему-то это не работает как должно. Возможно, баг в самом компоненте?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant