Skip to content
OverviewSolutions
16
Comments
6
Sammy
@soyed

All comments

  • Sammy•280
    @soyed

    hi @TejaswiniLabade Great job completing the challenge 🎉!

    Some Feedback:

    • I would recommend resetting the form if there are no errors after a user clicks submit.

    I hope that helps! 😊

    Cheers, Sammy

    Marked as helpful
  • Sammy•280
    @soyed

    Hi @Naiforsyth Good job! completing this challenge

    Great Questions by the way

    some feedback:

    1. Yes, CSS Grid can be used to create this component.
      • CSS Grid is used to setup layout for your page or for larger components of your website
      • Flexbox can be used to style the content of your grid items or smaller components
        • An example, The QR Code and the content of the card can be styled using Flexbox
      • I would recommend reading this article
    2. For responsive using rem and % works fine. This is definitely a tricky one because these units have different use cases depending on what you are trying to accomplish.
      • I would recommend reading this article

    I hope that helps! 😊

    Cheers, Sammy

    Marked as helpful
  • Sammy•280
    @soyed

    Hi @matheuspergoli Good job!

    Feedback

    • Optional but I would recommend resetting the form if there are no errors after filling all the input fields.

    I hope that helps!! 😊

    Cheers, Sammy

  • Sammy•280
    @soyed

    Hi @whiplink Good job, your solution looks great!

    Some Feedback:

    • I noticed your images are not displaying, I would recommend fixing it by providing a relative path to your images Relative path in HTML

    I hope this helps! 😊 Cheers, Sammy

  • Sammy•280
    @soyed

    Thank you 😊 I totally missed that

  • Sammy•280
    @soyed

    Hi @pawel975 This looks good! Just a couple of things I would add

    • Add cursor: pointer on the header items
    • Add cursor: pointer on the delete icon in the cart as well
    • one more thing I noticed was you had several ids # attached to your CSS. may I ask why you opted for that?
    • Another thing I noted on Firefox is the page layout looks different. I do not see a modal for the images.
    • Also, remember to make the accessibility changes as well

    But great job overall. Keep grinding! 🤝

    Marked as helpful