Order Summary Page

Community feedback

• 

  P

  Alexander Lippitsch•120

  @LipAlex1Posted over 2 years ago

  Hi Chitrang,

  you may want to look at the report as there are ally issues. Make sure you use semantic html (main, header, nav, section, aside, footer etc.). Assigning classes to main, header and footer tags is redundant code as these tags are meant to be used non-repetetively.

  Also, make sure you use ONE h1 tag since this is important for SEO reasons and it's the quasi standard in webdev.

  The background doesn't look like the specs. There is more work to be done on background image and background-colors to get it to look right.

  The site is not fully responsive particularly because there are no breakpoints in the design and the font-sizes are set to fixed widths that hurt the design on smaller viewport widths.

  Designing the entire HTML structure with divs only is considered bad style but not necessarily wrong. Semantic tags as well as button, input and list tags would make that code leaner, faster and easier to maintain. Commenting the code is doing yourself a favor as it saves you time when revisting the code.

  Keep up the good work and happy coding!

  Marked as helpful

  1

  ---

  

  Chitrang•150

  @cwebdevPosted over 2 years ago

  @LipAlex1 Thank you for the feedback! I will look into it.

  0