How’s it going, I'm a Adam, a Full-Stack Engineer, actively searching for work. I'm all about JavaScript. And Frontend but don't let that fool you - I've also got some serious Backend skills.
Location
City of Bath, UK 🇬🇧
Education
10 plus years* active enterprise development experience and a Fine art degree 🎨
Just a few little thing really, I'm all about details.
There are some let's that should be const because you don't change the values of them, I use way more const than let usually.
Perhaps more destructuring would be great.
I would also not use uppercase variables for the window object literals. Uppercase is is generally used for constants.
The try catch, not sure that's doing much because errors will be reported either way and there is no fallback logic anyway to handle a fault.
The comments could consolidated to reside in the jsdoc block with a @description, you might not need to explain as much with the comments, line by line.
That's mostly opinion and subject to code style and reasons I'm not aware of 😋
Agree that there are probably too many miscellaneous let's lying around.. For the window objects I guess I see them as constants in some ways - as in they are used throughout the application and shouldn't be changed, however they are appended to so perhaps uppercase is not appropriate.
The try catch is more to stop the whole thread crashing (known potential other issue..) if there is an error within a function handler. And with the comment's style it was more for the purpose of this article rather than for production, but none the less a good point.
Really appreciate the feedback, will keep you posted if we manage to open source it ✨
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
Just a few little thing really, I'm all about details.
There are some let's that should be const because you don't change the values of them, I use way more const than let usually.
Perhaps more destructuring would be great.
I would also not use uppercase variables for the window object literals. Uppercase is is generally used for constants.
The try catch, not sure that's doing much because errors will be reported either way and there is no fallback logic anyway to handle a fault.
The comments could consolidated to reside in the jsdoc block with a @description, you might not need to explain as much with the comments, line by line.
That's mostly opinion and subject to code style and reasons I'm not aware of 😋
The 😈 is always in the details..
Agree that there are probably too many miscellaneous let's lying around.. For the window objects I guess I see them as constants in some ways - as in they are used throughout the application and shouldn't be changed, however they are appended to so perhaps uppercase is not appropriate.
The try catch is more to stop the whole thread crashing (known potential other issue..) if there is an error within a function handler. And with the comment's style it was more for the purpose of this article rather than for production, but none the less a good point.
Really appreciate the feedback, will keep you posted if we manage to open source it ✨