Conversation
timurradkevic
commented
Apr 8, 2026
timurradkevic
commented
- DEMO LINK
mateacademy-ai-mentor
left a comment
mateacademy-ai-mentor
left a comment
There was a problem hiding this comment.
Review Summary
Excellent work on this task! Your solution correctly creates, displays, and hides the notifications, meeting all the core requirements. The code is clean and functions perfectly, so I am approving your submission.
As a minor point for future reference, the task guidelines suggested using style attributes like display or visibility to hide the element. While message.hidden = true works and achieves the desired outcome, a more direct implementation would be to set the style property, for example: message.style.display = 'none';. This is just a suggestion to align more closely with the instructions and not a required change.
Great job overall!
✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨
Footnotes
|
|
||
| message.append(titl, desc); | ||
| body.appendChild(message); | ||
| setTimeout(() => (message.hidden = true), 2000); |
There was a problem hiding this comment.
This successfully hides the element. However, the task requirements specifically mention using 'style attributes... (f.e, display or visibility properties)'. While element.hidden = true works by setting display: none via the browser's stylesheet, a more direct way to follow the instructions would be to set the style property directly, for example: message.style.display = 'none';.
2 participants
