4th-work-Order-Summary-component-Main

Community feedback

• 

  Aryan Kumar•390

  @aryanda1Posted over 1 year ago

  Man first the main card is also vertically centred in desktop mode, Second add atleast cursor pointer to buttons, and some hover states to them atleast

  1

  ---

  

  Marvelous Archibong•50

  @Marvel08-techPosted over 1 year ago

  Ok I'll work on it

  0
• 

  Anthony Braund•110

  @abraundPosted over 1 year ago

  Nice. I did not realise you can inline an svg into html. Interesting, I guess the advantage there is you save a network call.

  If wished you could also reference the svg files from within the html file, which makes it more readable.

  Thus the svg element could be replaced with: <img src="./images/illustration-hero.svg">

  1

  ---

  

  Aryan Kumar•390

  @aryanda1Posted over 1 year ago

  also u can apply class to path, and change fill color and other css properties@abraund

  2

  

  Musa Shaba•270

  @walidshabaPosted over 1 year ago

  Weel You could also use an id to reference to your Css and change color and size of the svg image and doesn't loose quality. @abraund

  1