Skip to content
OverviewSolutions
12
Comments
8
Julien
@EdoardoRamondetti

All comments

  • Julien•150
    @EdoardoRamondettiPosted about 2 months ago

    good work

    0
  • Julien•150
    @EdoardoRamondettiPosted 2 months ago

    Congrats on completing this challenge it looks good, but i notice that the background color is wrong and the image sizes should be revised

    0
  • Julien•150
    @EdoardoRamondettiPosted 2 months ago

    Amazing work

    0
  • Sunil•180
    @SunilParbhooSubmitted 3 months ago
    What are you most proud of, and what would you do differently next time?

    This was my first project attempting to build from a mobile first perspective and then shifting to desktop vs. building desktop styles first. I feel the shift in perspective was easier to make than I anticipated and look forward to building more projects utilizing this method in the future.

    What challenges did you encounter, and how did you overcome them?

    This project was pretty straight forward. One small issue that does come to mind was trying to get the text to line up the way it was in the design. I found that using text-wrap: balance made the text line up in the respective element nicely and also matched the designs.

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

    I am open to any general constructive critique that can make my code look and run better, especially if it is regarding best practices going forward. Thanks in advance!

    Product Preview Card HTML CSS

    1
    Julien•150
    @EdoardoRamondettiPosted 3 months ago

    awesome it looks really good

    0
  • Julien•150
    @EdoardoRamondettiPosted 3 months ago

    Great job

    0
  • Julien•150
    @EdoardoRamondettiPosted 3 months ago

    I really like how you have set the css and html code and this solution is very good but i notice that there is slightly more space at the end of the <main> maybe you should decrease the padding-left of the <main>

    0
  • Julien•150
    @EdoardoRamondettiPosted 3 months ago

    I don't see your code but the "Pubblished 21 Dec 2023" should not have the same color of the <p> and the text near the icon seems have a wrong value of font-weight but other than that is good

    0
  • alexpeteronoja•80
    @alexpeteronojaSubmitted 3 months ago
    What are you most proud of, and what would you do differently next time?

    I am happy to complete the project.

    What challenges did you encounter, and how did you overcome them?

    centering a div and i was able to do it.

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

    Please Review it and tell me where to improve

    QR code

    1
    Julien•150
    @EdoardoRamondettiPosted 3 months ago

    you should have inserted a margin-bottom to the div with class "attribution" to make bigger the qr code bigger,but other than that everything is ok

    0