search

IsabellaSoares / Bootcamp-React-WomakersCode

1 stars 0 forks source link

Indentação #8

Open lalizita opened 4 years ago

lalizita commented 4 years ago

https://github.com/IsabellaSoares/Bootcamp-React-WomakersCode/blob/40ee48c3c15967760ed67fa2625675c557f45e30/cep-api/src/form/AddressForm.js#L47

Primeiramente, parabéns pelo projeto do CEP! vi que usou bastante hooks!

Padrão de indentação e escrita de código é bastante importante e ferramentas como Prettier e Eslint ajudam bastante nisso.

Como padrão recomendo alterar a indentação do arquivo para 2 espaços, pois é melhor para se evitar um scroll horizontal, pense em quem dará manutenção nesse código algum dia, afinal passamos mais tempo lendo o código do que escrevendo.

IsabellaSoares commented 4 years ago

Oi, Laís! Não conhecia essas ferramentas, muito obrigada pelas dicas! Também não sabia sobre o padrão de indentação, já vou adotar 😄 Fiz as correções usando os atalhos do Prettier, pode dar uma olhada e me dizer se ficou certinho agora, por favor? Tenho dúvidas sobre quando se torna necessário colocar as propriedades dos componentes cada um em uma linha, ou quando posso deixar tudo inline mesmo. Posso usar como base a necessidade do scroll horizontal pra avaliar a melhor opção?

lalizita commented 4 years ago

Você consegue abrir PR mencionando essa issue? fica mais fácil de visualizar a correção

Sobre a duvida das propriedades de componentes, o acredito que quando tem mais de 3 props já é melhor quebrar a linha, acredito que o lint ja faça isso