Repository navigation
solution - #1912
solution#1912smialeknicole wants to merge 1 commit into
Conversation
mateacademy-ai-mentor
left a comment
There was a problem hiding this comment.
Review Summary
Your solution is approved and meets all task requirements. The code correctly generates table rows from the people array with all 6 required cells (name, gender, born, died, age, century), calculates age as died - born, calculates century using Math.ceil(died / 100), finds the dashboard table with document.querySelector('.dashboard'), and appends the rows. The gender conversion from 'm'/'f' to 'Male'/'Female' is also working correctly. The only minor improvement would be renaming the variable Nazwy (Polish for "names") to an English name like rowData or cells for better code clarity, but this is a cosmetic issue that doesn't affect functionality.
✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨
DEMO LINK