Skip to content
Submitted almost 2 years ago

Perfume card

BoΕΌenaβ€’100
@b-pieta
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Product preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hello everyone!

This is my first project after a week of learning CSS and HTML. I had some issues at first, especially with image control but at the end, it's not that bad. What is worth changing? Cheers πŸ¦‘.

Community feedback

  • バレンタむン πŸ˜ˆβ€’64,190
    @VCaramesPosted almost 2 years ago

    Hey there! πŸ‘‹ Here are some suggestions to help improve your code:

    • The alt tag description for the image needs to be improved upon. You want to describe what the image is; they need to be readable. Assume you’re describing the image to someone.

    More Info:πŸ“š

    https://www.w3.org/WAI/tutorials/images/

    • There is no need for position absolute/ relative to position the image. Using flex will do.
    • Implement a Mobile First approach πŸ“± > πŸ–₯

    With mobile devices being the predominant way that people view websites/content. It is more crucial than ever to ensure that your website/content looks presentable on all mobile devices. To achieve this, you start building your website/content for smaller screen first and then adjust your content for larger screens.

    More Info:πŸ“š

    Mobile First

    If you have any questions or need further clarification, feel free to reach out to me.

    Happy Coding!πŸŽ„πŸŽ

    1

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord