fix: Add patch fix for firefox mouse wheel #49
Conversation
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/react-component/virtual-list/kaafgvlat |
Codecov Report
@@ Coverage Diff @@
## master #49 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 13 14 +1
Lines 345 357 +12
Branches 60 63 +3
=========================================
+ Hits 345 357 +12
Continue to review full report at Codecov.
|
| onWheelDelta(offsetRef.current); | ||
| // Patch a multiple for Firefox to fix wheel number too small | ||
| // ref: https://github.com/ant-design/ant-design/issues/26372#issuecomment-679460266 | ||
| const patchMultiple = isMouseScrollRef.current ? 10 : 1; |
There was a problem hiding this comment.
和 FF 内部的 mousewheel.default.delta_multiplier_y 有关，FF 本身不推荐修改它，可以忽略。如果遇到了再调整：
| @@ -0,0 +1,2 @@ | |||
| const isFF = /Firefox/i.test(navigator.userAgent); | |||
There was a problem hiding this comment.
Need to fix for serveer-side-rendering
const isFF = typeof navigator !== 'undefined' && /Firefox/i.test(navigator.userAgent);Current nodejs error:
ReferenceError: navigator is not defined
vpc-admin: [dev] at Object.<anonymous> (/Users/nodkz/www/ps/vpc/apps/admin/node_modules/rc-virtual-list/lib/utils/isFirefox.js:7:28)
|
@zombieJ you break server-side-rendering on nodejs |
|
Hotfix #50 |
resolve ant-design/ant-design#26372
Firefox 的鼠标滚动触发的 wheel 事件
deltaY会比触摸板小很多。导致滚动看起来只有一小段距离，加一个 patch 来监听 FF 事件，如果发现 wheel 和 滚轮 事件数值相同（说明用的是鼠标），则添加一个倍数来滚动。自己这边测了一下，FF 鼠标滚动和滚屏大概差 8 ~ 11 倍之间