add task solution#2283
Conversation
PrimaSB
commented
Jun 19, 2026
- DEMO LINK
mateacademy-ai-mentor
left a comment
There was a problem hiding this comment.
Review Summary
Your solution is approved because it correctly fulfills the core task requirements and there are no critical issues blocking functionality. In src/scripts/main.js you successfully select all span.population elements, normalize their text values by removing thousands separators, convert them to numbers, and compute both the total and average. You then insert these computed values into the .average-population and .total-population elements, using toLocaleString() so the output keeps appropriate thousands separators. The README/demo link has also been updated to use your actual GitHub username instead of the placeholder, which aligns with the task requirement. As a minor improvement (not required for this task), you could consider using a more explicit conversion like Number() and slightly more descriptive variable names to make the code even clearer for future readers. Overall, you met the functional and implementation constraints very well—nice work.
✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨