manage-landing-page-master

Community feedback

• 

  BrunoMoleta•700

  @brunomoletaPosted 6 months ago

  Hello NiLoBol,

  Please check the Pull Request at Github. I reviewed specially the html and Vue architecture of the project.

  I divided the components into smaller parts and refactored your Button component so there was no repetition, and it's easier to read.

  Please check the commits to understand the HTML semantics I put in there. You only used divs, which carry no semantic meaning. I changed them to form, section, header, main, and footer.

  Also, I removed the raw strings from the components and put it all in .json files.

  The hook you created for the Testimonials is neat. I had not yet seen that, congratulations.

  Best regards from 🇧🇷 :)

  Marked as helpful

  1