Closed tvgc closed 2 years ago
Bom dia!
Recebido sua mensagem. Quando você liberar para fazer a merge eu fecharei esta issue.
pode fazer a merge...
Bom dia!
Fiz a merge e não houve nenhum confito.
Mas gostaria de sugerir alguns detalhes para mudança:
Acredito que o título do aside se aproximaria mais do template original caso houvesse a tag
entre as palavras "your" e "skills".
Seria interessante a utilização das classes reutilizáveis ao máximo, uma vez que auxiliaria na mudança do código de maneira mais rápida e fluida. Uma sugestão seria trocar a classe footer__header-title por title-level-3 com a utilização da tag h3, pois se aproxima mais do padrão das demais sessões. Daí, so restaria trocar a cor. Flataria ainda a troca da classe container pela classe reutilizável criado.
Por fim, gostaria de dizer que criei duas classes para deixar o icone chevron maior e centralizado. como estava: como ficou:
estarei arrumando assim que possivel (acredito que amanhã)... Estava focado na entrega de um exercício do curso.
Feito as alterações
Verificado código e feito merge.
Ficou muito bom; e também gostei da animação no botão chevrom.
Boa noite Thales, deixei uma classe chamada :
Foi pra poder ver como ia ficar com o tamanho real da pagina, pode tirar ela quando for fazer o merge.