"too much recursion" in https://unpkg.com/mobius1-selectr@latest/dist/selectr.min.js

Mobius1 / Selectr

A lightweight, vanilla javascript select box replacement. No dependencies.

MIT License

313 stars 77 forks source link

Open trailsnail opened 5 years ago

trailsnail commented 5 years ago

Hi there,

following problem:

call https://codepen.io/Mobius1/pen/QgdpLN
change "Native Dropdown (Single)"
see console for error too much recursion selectr.min.js:46:192
replace codepen setting JS to https://unpkg.com/mobius1-selectr@latest/src/selectr.js the error is gone away

Maybe it's another reason like this one: grafik but if not, the error should be researched

Kind regards

@Mobius1 Can you tell me about your built process, how the dist/*min files are created?

trailsnail commented 5 years ago

It seems, that Selectr instance executes and triggers its own change listener in a loop

dowadidi commented 4 years ago

@trailsnail I had the same issue (only when rendering on mobile) and fixed it using the non-minified version (with webpack).

trailsnail commented 4 years ago

@dowadidi Thanks, and sure - we did this too. But the problem must be solved anyway :smirk: