Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.
And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.
_handleArrowSelect function - uncaught reference (i) in for loop
See original GitHub issueWhen I do any search and try to use the arrow key to navigate the results i receive a reference exception:
Uncaught ReferenceError: i is not defined
This is triggered by the for loop on line 828 in leaflet-search.src.js. i is an implicit global variable and in strict mode this will kick up an error:
for (i=0; i<searchTips.length; i++)
If you declare i then it fixes the error. It becomes a local variable i.e:
for (let i=0; i<searchTips.length; i++)
Can this be updated in the package?
Issue Analytics
- State:
- Created 5 years ago
- Reactions:1
- Comments:7 (4 by maintainers)
Top Results From Across the Web
Top Related Medium Post
Top Related StackOverflow Question
Troubleshoot Live Code
Top Related Reddit Thread
Top Related Hackernoon Post
Top Related Tweet
Top Related Dev.to Post
Top Related Hashnode Post
Dude, just updated to the 3.0.1 version and double checked the GitHub code, the line
for (i=0; i<searchTips.length; i++)is still there, if you replace it withfor (let i=0; i<searchTips.length; i++)as @Robc24 mentioned, the up/down arrow key function works again.https://www.npmjs.com/package/leaflet-search/v/3.0.0
https://github.com/stefanocudini/leaflet-search/commits/v3.0.0