MAX_LENGTHは固定じゃなくてiniから算出したほうがいい？

utubo / firefox-simple_gesture

🦊An add-on that adds simple touch gestures for Firfox for Android. Sorry, I only check issues sometimes.

https://addons.mozilla.org/ja/firefox/addon/simple-gesture/

Other

32 stars 1 forks source link

MAX_LENGTHは固定じゃなくてiniから算出したほうがいい？ #14

Closed utubo closed 6 years ago

utubo commented 6 years ago

タイムアウト的にMAX_LENGTHに達することはなさそうだし、算出する分ちょっと遅くなるかな？ iniロード後

maxLength = 0; for (let g in SimpleGesture.ini.gestures) { if (maxLength < g.length) maxLength = g.length; }

touchmoveでmaxLengthに達した場合

if (gesture.length >= maxLength) { if (SimpleGesture.onMaxLength && !SimpleGesture.onMaxLength(e)) return false; // don't reset, when edit gesture. resetGesture(); return false;

utubo commented 6 years ago

複雑にする必要なしやめやめ