product preview solution

Community feedback

• 

  Lucas 👾•104,400

  @correlucasPosted over 2 years ago

  Hello Buci!

  Congratulations for your solution!

  I've some considerations about your html structure.

  1.Note that you don't need 2 containers to hold the two div correspondent to the two columns, use only the main content to keep this elements.

  2.If you work with percentage is hard to know how much big is the container, it should be around 750px/800px, I think percentage is better for the elements inside the container, so try to use a value around .container {max-width: 800px;} and the elements will have the percentage in base of this width.

  3.Use flex-wrap for a better behavior in the price section, this allows the section to break the line automatically, .des-container .price {flex-wrap: wrap}

  I hope it helps, keep coding!

  1

  ---

  

  Buci•40

  @alphardhafizPosted over 2 years ago

  @correlucas WOW thank you so much for the feedback, i'll fix this and remember the second point that better do not use percentage for container

  1