emadmri / javascript Goto Github PK

You forgot to add an ID to the list element. Due to this, you will not be able to get the list elements by ID (while using the getElementByID function).

You forgot to make a CSS file to style the HTML document.

In this function, you have used var to declare 'img' instead of let. Due to this, the 'img' is available globally. Now, if you see on line 125, you have appended the 'img' to body element after already appending it the li element. But each element can have only one parent, hence the 'img' is removed from the li element and is added to the body. Therefore, when your loop will run successfully, the last 'img' in the data would be added to the body instead of the li element.

If you had used let instead of var, the scope of the 'img' would have been limited to the loop and the statement on line 125 would have given an error (which you can check on the console in the web browser).

You have added just one image and duplicated it for others as well. This way, you won't be able to add images to elements having ID other than 'mind_for_number'.