Skip to content
Submitted about 2 years ago

Responsive Product Card - SASS, CSS3 and HTML5

Charles Barros•70
@charbavito
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

This is my proposal of solution to the Product Card Challenge and was my third non-tutorial project in frontend carrer. As the two first, this challenge consists of creating a simple static page using the jpg design file as reference and implementing only HTML5 and CSS3, but in this challenge I had to use new responsiveness concepts such as Media Query in CSS, because the Challenge's Style Guide asked me for two different layouts, one to Desktop and other to Mobile. Again a project with great learning, where I was able to apply and strengthen all the knowledge acquired with the previous challenges and add new important knowledge about SASS pre-processor, use of partials and responsivness with Media Queries.

Community feedback

  • バレンタイン 😈•64,190
    @VCaramesPosted about 2 years ago

    Hey there! 👋 Here are some suggestions to help improve your code:

    • The purpose of the Main Element is to identify the main content of your page. It is not the container of you component. After the main element, you want add a container to wrap you separate components in.

    • 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.

    • This challenges requires the use of two images 🎑 for different breakpoints. The Picture Element will facilitate this.

    Here is an example of how it works: EXAMPLE

    Syntax:

      <picture>
    <source media="(min-width: )" srcset="">
    <img src="" alt="">
  </picture>

    More Info:

    https://www.w3schools.com/html/html_images_picture.asp

    https://web.dev/learn/design/picture-element/

    • The only heading in this challenge is the name of the perfume, “Gabrielle Essence Eau De Parfum” . The rest of the text should be wrapped in a Paragraph Element.

    • The old price 🏷 is not being announced properly to screen readers. You want to wrap it in a Del Element and include a sr-only text explaining that this is the old price.

    If you have any questions or need further clarification, let me know.

    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