Baegan-189• 90
@Baegan-189
Submitted
Adam
@apekul
All comments
- Adam• 200
@apekul
Posted
Nice work! You can improve your project by implementing semantic HTML tags like <section>, <article>, <header>, and <main> to enhance code readability. Adding a min-height property to a global container, such as the <body> or a main wrapper <div>, ensures that the content takes up the full height of the viewport. Additionally, using display: flex on this container will help center content both horizontally and vertically, contributing to a more visually appealing design.
0 - PDan Marius• 625
@danmlarsen
Submitted
What are you most proud of, and what would you do differently next time?
Any feedback is appreciated.
Adam• 200@apekul
Posted
Well-structured code with effective use of semantic tags. A great solution to the challenge.
1 - Syed-1129• 50
@Syed-1129
Submitted
Adam• 200@apekul
Posted
Nice work! To enhance your solution further, consider using flexbox for improved content centering and alignment control. Incorporating semantic HTML tags will enhance readability and improve the structure of your code. Lastly, adding hover transitions on anchors will introduce smooth visual effects, making interactions more engaging and user-friendly.
Marked as helpful
0 - PMohd Sadaf• 460
@MsadafK
Submitted
Adam• 200@apekul
Posted
The code looks pretty good overall. I would recommend using semantic HTML tags.
1 - Shid• 10
@shid05
Submitted
Adam• 200@apekul
Posted
The code is pretty good, and the solution design, though slightly different from the task specifications, is acceptable. However, I would recommend keeping the styles in a separate file as it is considered best practice. This approach enhances readability, maintainability, and reusability of the code.
0
