Skip to content
OverviewSolutions
2
Comments
2

Karolina Pietraś

@ilieretteWarsaw40 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

  • Submitted 10 days ago

    Blog preview (with multiple tags)

    #bem#accessibility
    • HTML
    • CSS
    1

    I don't know - it seemed easy.

  • Submitted 28 days ago

    QR Code

    • HTML
    • CSS
    3

    I struggle to understand where use px, em and rem and about accessibility.

Latest comments

  • P
    Karolina Pietraś•40
    @ilierettePosted 10 days ago

    Looks good! Good job with using custom variables in css.

    I noticed is lack of borders of card component. It have shadow, but also borders. That's a minor style issue.

    Also I might be wrong, but I don't know if "article" is good semantic tag for it - it's more like card component than article. I would use main and put inside div with class card.

    Marked as helpful
    0
  • P
    Karolina Pietraś•40
    @ilierettePosted 28 days ago

    Hello! First feedback there, so take it with a grain of salt. Overall looks good and very similar to project, good job! But was the background choice deliberate? It's grey on design. And divs are not semantic tag. That's all, keep up!

    0