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! Your solution is approved. You've done a great job decomposing the application into separate components and correctly co-locating the CSS files with their respective components. The project structure is clean and meets all the requirements. The way you handled the Header component is perfectly acceptable; creating a separate Navigation component was optional, and your implementation is solid. Keep up the great work!
✨ 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.
The task description suggests that the navigation could be its own component. If you were to do that, these navigation-related styles would move to a Navigation.css file. For this task, keeping them here is acceptable since the navigation is part of the Header component.
2 participants
https://o-koszalka.github.io/react_decompose/