Skip to content
OverviewSolutions
3
Comments
3
P

Matej

@matej-t90 points

I’m a mysterious individual who has yet to fill out my bio. One thing’s for certain: I love writing front-end code!

Latest solutions

Latest comments

  • P
    Matej•90
    @matej-tPosted 26 days ago

    For multiple instances of a same element, like the links here, instead of just using a div wrapper, for accessibility you should use a list (ordered or unordered) with list items, and then place the links in each list item.

    Also, because the links here look like buttons, you can set role="button" on the <a>

    Marked as helpful
    0
  • P
    Matej•90
    @matej-tPosted 30 days ago

    You have not achieved the challenge's main requirement, which is the hover states (the title should become yellow when hovered and have a pointer).

    Try to improve the elements grouping using semantic elements like <main> and <section>.

    Also, the sizes of the card and image don't quite align with the provided preview image. Maybe try matching closer.

    Also, you can achieve the card shadow by using box-shadow in CSS. There is no need for ::after.

    Look into CSS variables. Look into Flexbox, and how a flex container controls the size limitations of children components.

    Marked as helpful
    0
  • P
    Matej•90
    @matej-tPosted about 1 month ago

    Maybe try box-sizing: border-box in order to have more control over the sizes of elements.

    Also, some margin/padding on the top of the attributions would help visually.

    Marked as helpful
    1