add task solution #4251
add task solution #4251
Conversation
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! I am approving your solution as you have successfully met all the core requirements. You've done a great job decomposing the application into Welcome, Header, and Article components and correctly splitting the corresponding CSS into separate files.
For future improvement, consider the suggestion to create a separate Navigation component. While this was optional, extracting the navigation markup from Header.jsx and its styles from Header.css into their own dedicated files would further enhance the modularity of your application. Overall, this is a very solid submission. Keep it up!
✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨
Footnotes
| .navigation__link { | ||
| display: inline-block; | ||
|
|
||
| padding: 0 0.5em; | ||
|
|
||
| color: black; | ||
| text-decoration: none; | ||
|
|
||
| opacity: 0.7; | ||
| outline: none; | ||
|
|
||
| transition: opacity 0.2s ease-in-out; | ||
| } | ||
|
|
||
| .navigation__link:hover, | ||
| .navigation__link:focus { | ||
| opacity: 1; | ||
| } |
There was a problem hiding this comment.
These navigation styles are tied to the <nav> element inside Header.jsx. The task description suggests creating a separate Navigation component. Moving these styles into a Navigation.css file would improve the component structure and modularity of your application.
2 participants
DEMO LINK