You are viewing a single comment's thread from:
RE: [ E-learning Js 0.1 ] Release 0.1.3 - alpha 1
Thank you for your contribution.
- magik numbers such as
tempPage <= 9 - You might want to let users define these data in a JSON/XML instead of
main.js - you probably could use some librarys/frameworks to make your Javascript code such as
document.getQuerySelectoror.style.display=shorter. - some code duplications (could be refactored) for example:
const questionsHtml = document.querySelector(options.questionsHtml);
const resetButton = document.querySelector(options.resetButton);
...
Your contribution has been evaluated according to Utopian policies and guidelines, as well as a predefined set of questions pertaining to the category.
To view those questions and the relevant answers related to your post, click here.
Need help? Chat with us on Discord.
Thanks @justyy for your recomendations, I will take this improvements in the project dashboard in github.
Thank you for your review, @justyy! Keep up the good work!