single price grid component but with flex

Community feedback

• 

  Mo•840

  @MohamedAridahPosted over 2 years ago

  Hello @Gab-Ferreira, You did a very good job for this solution, yours solution also is Responsive. So will done👏🎉.

  However i have some notes for you:

  • it was very obvious that padding and box-shadow were too much. Try to reduce the padding for header, .sign-up and .why-us elements.

  	.card {
  		box-shadow: 0px 8px 22px 5px rgb(0 0 0 / 12%)
  	}
  

  • instead of using <button> element for sign up. you better use <a> element for this because this action may lead to separate page to complete the sign process.

  • .texts-whyus instead of using <div> and br. you should use unordered list <ul> instead.

  	<ul class ="texts-whyus">
  		<li>Tutorials by industry experts</li>
  		<li>Etc...</li>
  	</ul>
  

  I hope this wasn't too long for you, hoping also it was useful😍.

  Goodbye and have a nice day. Keep coding🚀

  Marked as helpful

  0

  ---

  

  GabFerreira•240

  @Gab-FerreiraPosted over 2 years ago

  @MohamedAridah I definitely have to improve, thanks a lot.

  0

  

  Mo•840

  @MohamedAridahPosted over 2 years ago

  @Gab-Ferreira you are welcome🌹👍

  0