product-preview-card-component

Community feedback

• 

  Kevin Tatagiba•270

  @kevintataPosted about 2 years ago

  Hey man, Congrats on the solution!

  I'd recommend checking out flexbox, it'll make this project much easier, put both cards into one container (div) Then set the div to

  div {
        display:flex;
        justify-content: center; 
        align-items: center; 
        flex-direction:row; 
  }
  

  and boom perfectly aligned!

  Marked as helpful

  1
• Account deleted

  Hello @Gehadmo2,

  I have some tips to improve your solution:

  • Remove the pink background color and add the cream color to body
  • Be sure to reference your style-guide.md file, colors and fonts are given to you
  • Images must have alternative text applied to them
  • Every document must have one main landmark instead of using a <div use a <main>
  • More info about HTML5 Elements
  • Here is my solution

  I hope these tips help you and good luck

  Marked as helpful

  0
• 

  Lucas 👾•104,400

  @correlucasPosted about 2 years ago

  This comment was deleted about 2 years ago

  0

  ---

  

  Gehad•70

  @Gehadmo2Posted about 2 years ago

  @correlucas do you write the same comment to everyone

  1

  

  Lucas 👾•104,400

  @correlucasPosted about 2 years ago

  @Gehadmo2 @Gehadmo2 Its true, I was trying to help you, but seems that you don't need it. Keep it up and keep posting amazing content. I'll make sure to never comment again any of your solutions to don't bother you again my friend.

  Have a nice day Gehad =)

  0