Fylo landing page two column layout

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 3 years ago

  👋Hi robjuh!

  For the curved background, you can use background-image and position it using background-position.

  For the feedback:

  • Use HTML semantic tags, like for instance, for every row, you can use section instead of div.
  • The class name should be more specific. Like, for example you can change the first row class into hero or showcase. It will make you understand what's going on in the future if you need this project as Portfolio or anything.
  • Try to use Prettier to format your code. You can install it as an extension from your text editor or you can install it through command line. Make sure that Node.js has already installed on your machine.

  npm i -g prettier
  

  • I recommend to use p instead of headings tag for this chunk of code, since they are just a name and a job title (not a title of a section).

  <h3>Kyle Burton</h3>
  <h4>Founder & CEO, Huddle</h4>
  

  • Use rem units instead of px, since rem unit will allow the user to control the size of your page.

  That's it! Hopefully this is useful!

  2