Add parse list DOM solution #1549
Conversation
There was a problem hiding this comment.
Great job on implementing the sorting functionality! 🎉 The code meets the core requirements of the task, and the sorting logic is correctly implemented. However, consider refactoring your code to define the sortList and getEmployees functions as specified in the task description. This will make your code more modular and easier to maintain. Keep up the good work! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| const employees = [...document.querySelectorAll('li')]; | ||
|
|
||
| const sortedEmployees = employees.sort( | ||
| (a, b) => parseSalary(b.dataset.salary) - parseSalary(a.dataset.salary), |
There was a problem hiding this comment.
The sorting function currently sorts the employees in descending order based on their salary. Ensure that the data-salary attribute is correctly set on each li element in the HTML, as this script relies on it to function correctly.
No description provided.