All comments

• 

  George Bleyer• 280

  @georgebleyer

  Submitted over 1 year ago

  Hello DEVS

  In this project I had some difficulties.

  1. Center the <div class="container"> in the center of my page, as I wasn't able to center it, I put a margin and it stayed like this;
  2. For the responsive site it was ok but I thought that for the cell phone it was a little small;
  3. I couldn't change images so I just fixed the height and width of it.

  If you have any tips, leave them in the comments.

  

  Product preview card component

  2

  40

  

  Tiago Ribeiro• 80

  @TiagoRidev

  Posted over 1 year ago

  Hi George. Congratulations for the challenge!

  I would like to give you a suggestion to center your container, try to use these properties in css. .container{ display: flex; justify-content: center; align-items: center; flex-direction: column; height: 100vh; }

  Congrats for the project. I hope this feedback is helpful for you.

  Marked as helpful

  0
• 

  Rama• 70

  @ramavats

  Submitted over 1 year ago

  please let me know if I made any mistake.

  

  3-column-preview-card-component

  1

  71

  

  Tiago Ribeiro• 80

  @TiagoRidev

  Posted over 1 year ago

  Hello Rama, congratulations for the project.

  I have a tip for u, your mobile responsive is very good, maybe putting max-width 800px or next, the screen does not cut the informations. You can apply responsive always the screen to start cut the contents.

  Congratulations and keep up your good work!

  1

  This comment has received 1 replyShow