rust-lang/rust
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

rustdoc: Bind keydown instead of keypress for nav

Browse Source 
Apparently keypress doesn't quite work in all browsers due to some not invoking
the handler and jquery not setting the right `which` field in all circumstances.
According to http://stackoverflow.com/questions/2166771 switching over to
`keydown` works and it appears to do the trick. Tested in Safari, Firefox, and
Chrome.

Closes #15011
This commit is contained in:
Alex Crichton
2014-07-25 08:54:38 -07:00
parent 724bcec089
commit 80efb220e0
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/librustdoc/html/static/main.js
View File

@@ -357,8 +357,8 @@
}, 20); }, 20);
}); });
$(document).off('keypress.searchnav'); $(document).off('keydown.searchnav');
$(document).on('keypress.searchnav', function(e) { $(document).on('keydown.searchnav', function(e) {
var $active = $results.filter('.highlighted'); var $active = $results.filter('.highlighted');
if (e.which === 38) { // up if (e.which === 38) { // up