Product preview card component

Community feedback

• 

  Aung Ko Ko Khant•30

  @aungkokokhant16Posted over 2 years ago

  Thank u for u r advice. Next time I will care.

  0
• 

  Lucas 👾•104,340

  @correlucasPosted over 2 years ago

  👾Hello Aung Ko Ko Khant, congratulations for your first solution and 😎 welcome to the Frontend Mentor Coding Community!

  First at all, congrats I can say that this an amazing start to have the solution so good. Now you need only to work some paddings inside the text content section and you give this spacing with gap: 20px.

  And give the container size that in this case is max-width: 900px.

  👨‍💻 Here's my solution for this challenge if you wants to see how I build it: https://www.frontendmentor.io/solutions/product-preview-card-vanilla-css-and-custom-hover-state-on-hero-85A1JsueD1

  👋 I hope this helps you and happy coding!

  0
• 

  Illyaas•700

  @Illyaas4ShowPosted over 2 years ago

  Hi and welcome to the community!

  This is an amazing start to your journey. Your solution looks very nice :)

  I think you should increase the padding right of the container to allow the title and paragraph wrap more and look more like the design.

  I hope this helps, Happy coding!

  0
• 

  Abhishek choudhary•1,570

  @dazzlerabhi30800Posted over 2 years ago

  Your solution is good as well as code the only thing you need is to solve the accessibility issues by wrapping the whole component inside main tag.

  0