Skip to content
OverviewSolutions
8
Comments
5
Adam
@apekul

All comments

  • Adam•200
    @apekulPosted 4 months ago

    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
  • Dan Marius•885
    @danmlarsenSubmitted 4 months ago
    What are you most proud of, and what would you do differently next time?

    Any feedback is appreciated.

    HTML, CSS with flexbox

    1
    Adam•200
    @apekulPosted 4 months ago

    Well-structured code with effective use of semantic tags. A great solution to the challenge.

    1
  • Adam•200
    @apekulPosted 4 months ago

    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
  • Adam•200
    @apekulPosted 4 months ago

    The code looks pretty good overall. I would recommend using semantic HTML tags.

    1
  • Adam•200
    @apekulPosted 4 months ago

    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