@vanderms
Posted
Well done!
Your solution is very good and I loved it! My only tips are:
-
A extension like prettier (or other code formatter) could improve the readability of your code. Sometimes, I found the identation of your js solution hard to read.
-
Considering the risk of XSS atack, I wouldn't recommend using
innerHTML
, especially when the data is coming from the server. I probably didn't see all uses of innerHTML, but the cases that I saw you could replace it with the safer optionsinnerText
ortextContent
and the result would be the same. -
And your solution would be indeed more easy to understand if you had organized the files inside folders.
Keep up the good work!
Marked as helpful
@Marija-Kov
Posted
@vanderms Thanks a lot for the kind words and great tips!