Skip to content

solution#1824

Open
PatronXXi wants to merge 1 commit into
mate-academy:masterfrom
PatronXXi:develop
Open

solution#1824
PatronXXi wants to merge 1 commit into
mate-academy:masterfrom
PatronXXi:develop

Conversation

@PatronXXi
Copy link
Copy Markdown

- [DEMO LINK](https://PatronXXi.github.io/js_task_generate_table_DOM/)

Copy link
Copy Markdown

@mateacademy-ai-mentor mateacademy-ai-mentor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review Summary

Your solution is approved. All requirements have been successfully implemented: the people array is correctly converted to HTML table rows with all 6 required cells (name, gender, born, died, age, century), age is calculated as died - born, century uses Math.ceil(died / 100), rows are properly appended to the dashboard table, and no styles or HTML layout were modified. The demo link has been updated with your actual GitHub username. The code uses proper DOM manipulation and HTML table structure. Great work on meeting all the checklist requirements!


✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨

Footnotes

  1. Rate AI review example

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants