breatheco-de / student-client

BreatheCode Students Client (currently in development)
https://student-client.breathecode.now.sh
5 stars 10 forks source link

BUG: Timeline not scrolling properly with mousewheel on firefox #28

Closed jferragut closed 3 years ago

jferragut commented 3 years ago

Behavior: The issue is that scrolling is slow and janky to the point of frustration for users. Already had a few students complain. The following line appears to be the source.

https://github.com/breatheco-de/student-client/blob/7acde84586e0e1d4262a9ec791fc2fccdaed79db/src/js/components/menus/TimeLineMenu.js#L51

Expectation: In chrome, the scrolling is smooth and at normal speed. Firefox should be the same.

Browser: Firefox only OS: Linux, Mac, Windows 10

Reproduction steps:

jferragut commented 3 years ago

The current wheel handler and ref were the issue. To resolve, I removed the offending code. Scrolls perfectly in firefox and chrome now.

jferragut commented 3 years ago

Completed with PR #29