-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Util.js to es5 #58
Util.js to es5 #58
Conversation
Ignore debugging files and vscode config
I realised I could run util.js through chatGPT and ask it what wasn't valid ES5. It identified the use of |
I didn't need to roll back all the changes :-) Still learning. |
Thanks @dunxd - if we ever expand what's in util.js I would indeed consider splitting off an ES5 variant for very old e-readers |
Default values for function parameters were introduced in ES6. Old Kindle's Experimental Browser only supports ES5 features. I've changed the way that a default is handled, based on a method found on StackOverflow, and it works ok.
The initiateTwigged function that we moved earlier doesn't appear to use this, but I expect the issue is related to an unsupported language feature too.