FAQs

Community feedback

• 

  Bernardo Poggioni•5,360

  @R3ygoskiPosted 7 months ago

  Olá Filipe, parabéns pela conclusão do projeto.

  Gostaria de pontuar que o accordion não está funcionando da maneira correta, isso devido ao Flexbox, ele está fazendo com que os itens sejam mostrados lado a lado, para corrigir isso basta adicionar um flex-wrap: wrap; a sua classe .card, que assim o parágrafo será mostrado abaixo.

  Já a parte lógica está correta e muito bem feito, mas gostaria de dar uma dica, ao invés de usar o style, que faz com que você adicione CSS Inline, você poderia usar o classlist, que permite você alterar a classe de um elemento, e isso faz com que seja evitado o CSS Inline.

  Agora uma dica sobre o HTML, a estrutura do seu HTML está correta, mas notei que está com falta de semântica, por exemplo suas div.card poderiam ser <article> que isso já daria mais semântica, ou sua div.container poderia ser <section>.

  E era isso, novamente, parabéns pela conclusão do seu projeto, ele ficou muito bom, continue evoluindo. Caso algo que eu tenha dito não tenha ficado claro, por favor, comente aqui embaixo que tentarei ajudar da melhor forma possível.

  Marked as helpful

  1

  ---

  

  Filipe Barbosa•90

  @FilipeSBabosaPosted 7 months ago

  @R3ygoski Obrigado pelas dicas. Eu não sabia como resolver esse problema do parágrafo, mim ajudou muito. Sobre a parte semântica, eu ainda não sei muito o que devo usar, por isso uso muito <div> por ser mais genérico.

  1