search

Mottie / Keyboard

Virtual Keyboard using jQuery ~
http://mottie.github.io/Keyboard/
Other
1.78k stars 723 forks source link

kbcss is not defined in navigation extension #601

Closed Eschon closed 7 years ago

Eschon commented 7 years ago

I tried using the virtual keyboard with the navigation extension but if I use the F1 key to switch between navigation modes I get the following JS error: 

Uncaught ReferenceError: kbcss is not defined
    at $.keyboard.o.getMaxIndex (jquery.keyboard.extension-navigation.min.js?bc56:formatted:156)
    at $.keyboard.o.navigateKeys (jquery.keyboard.extension-navigation.min.js?bc56:formatted:156)
    at $.keyboard.o.checkKeys (jquery.keyboard.extension-navigation.min.js?bc56:formatted:156)
    at HTMLInputElement.eval (jquery.keyboard.extension-navigation.min.js?bc56:formatted:156)
    at HTMLInputElement.dispatch (jquery.js?eedf:5206)
    at HTMLInputElement.elemData.handle (jquery.js?eedf:5014)

I've looked into it and it seems like the minifier replaced kbcss in all places but one. This also only seems to be a problem in the npm version, the jquery.keyboard.extension-navigation.min.js file in the repository works.

Mottie commented 7 years ago

Hi @Eschon!

This was fixed in #588 and is available in the latest release.

Mottie commented 7 years ago

And... it looks like I might have forgotten to update npm. Done.

Eschon commented 7 years ago

Thanks for the quick fix! It works now.