tafaramafemba / leader_board Goto Github PK

1. Repeated variable declarations.
2. Leaving code that is not being used on the .js file

Good Points 👍

• Good naming practice.
• Clean HTML and CSS structure.
• Nice and simple UI.

To take note 📝

• To make a separation of concerns, it would be better to create one class for the tasks (data management) and another for the UI actions.
• Kindly organize your image assets in another folder to improve file structure.
• If you take your data array out of your constructor function and place it in the global scope, your code would be more organized, and the data will not be dependent on one object instantiation.
• Would look better if the input field would take 100% of the width of the list item.

• Code is very fine 👍

• Readable and easy to understand 💯

• Modularize the file api.js ::four_leaf_clover: