order summary component main

Community feedback

• 

  Benedikt Walch•0

  @ScienceMaster7Posted about 2 years ago

  Your Website is responsive which is great. but you could have made it a bit smaller so it looks more like the design. Speaking about semantic HTML you are definitely using too many divs, you could use header and footer tags instead for the first and last text and section tags in the middle instead of the divs. And for Structuring your code you did well in structuring your CSS classes from top to bottom but for later projects, it would be better to store your CSS in a style.css and not in the HTML file.

  Greetings and keep on coding.

  0

  ---

  

  Priyobroto Kar•260

  @PriyoBROTO04Posted about 2 years ago

  @ScienceMaster7 yes absolutely. Will try to keep all these things in my mind for future projects ... thanks a lot :)

  0