Stats-preview-card with flexbox

Community feedback

• 

  Thiago Correa•350

  @ThiagoBRG60Posted 11 months ago

  Oi !👋🏻

  O seu Stats Preview Card ta muito bom, aqui estão algumas sugestões:

  1 - Bom, acho que o principal que eu notei é que você usa css interno pra estilizar os componentes, no html mesmo. E mesmo que esse seja um projeto simples, recomendo criar um arquivo próprio de css mesmo, porque essa prática de css interno para projetos reais é um pouco complicada, e não muito recomendada, pois depois fica ruim para dar manutenção no código e etc, e também para deixar tudo mais organizado mesmo, e não acabar criando um arquivo html gigantesco, dependendo do tamanho do projeto.

  2 - Recomendo usar uma tag main dentro do body, pra indicar o conteúdo principal do seu projeto/site.

  3 - Um aviso que eu dou é cuidar com nomes, pensar em bons nomes para classes e ids é muito importante, para projetos reais e etc, seja mais objetivo nos nomes e tente evitar dar nomes genéricos, para que os outros também entendam seu código depois.

  4 - A última sugestão é deixar o seu site responsivo, usando media queries. Para deixar ele no tamanho exato nas telas menores, de celulares e tablets, pois percebi que em tela de celulares seu site vaza para as bordas. Media queries podem ser um pouco complicadas de entender no começo mas tenho certeza que rapidinho você pega o jeito.

  É isso, continue estudando e melhorando, e feliz ano novo! 🤝🏻

  Marked as helpful

  0

  ---

  

  João Pedro de Oliveira•120

  @jaojogadezPosted 11 months ago

  @ThiagoBRG60 Olá, muito obrigado pelo feedback, isso me ajuda a melhorar e tirou dúvidas minhas sobre o css interno e a tag main. Obrigado e feliz ano novo.

  0