Skip to content
OverviewSolutions
5
Comments
6
shaikh zayed
@undersigned-dev

All comments

  • P
    shruticodes01•270
    @shruticodes01
    What are you most proud of, and what would you do differently next time?
    • Using CSS Grid and Flexbox
    • Learning that button should can have a display of inline-flex instead of flex.
    • Using min-width to adjust image-size.
    • Using BEM naming method
    • Practicing to keep the code robust and readable.
    What challenges did you encounter, and how did you overcome them?

    Card image - width of 50% on card image and the class of card_details, ended up scaling the image down.

    But applying min-width on the image adjusted the image to 50% of the card width and 100% of the height.

    What specific areas of your project would you like help with?

    All kinds of feedback and different approaches to the solution would be helpful. Thank you.

    Product preview card with flexbox and grid

    2
    shaikh zayed•70
    @undersigned-dev

    It looks very well made. i liked that you declared some boiler plate css first like variable and box-sizing. I tried a different approach with less lines of css, But well done 👍

  • shaikh zayed•70
    @undersigned-dev

    Well done. you could have tried give nutrition fact content set width with inline-block element so they could have looked close to the design, but well done for what i can see

  • shaikh zayed•70
    @undersigned-dev

    site looks good on pc you should try making it mobile responsive as well

  • shaikh zayed•70
    @undersigned-dev

    well done for what i can see

  • shaikh zayed•70
    @undersigned-dev

    your site looks good. well done

  • shaikh zayed•70
    @undersigned-dev

    Hey there friend you could have added text-align:center; to your div with class named "box" and adjusted the img

    Marked as helpful