Anotações/Críticas construtivas:
- O arquivo HTML deveria estar fora da pasta SRC
- Os imports de css poderia ter criar um main.css por exemplo e dentro dele voce usasse o @import para importar todos os estilos em apenas 1 arquivo.
- O padrão de boas práticas para criação de nomes de arquivo é usando o hifen. exemplo: first-section.css
- escolher a linguagem a criar os nomes, classes etc. Voce acabou misturando em alguns momentos Ingles com portugues.
- Usar ID apenas se voce precisar acessar algo
- Senti falta de mediaquery para dispositivos menores, em especial mobile.
Anotações/Elogios:
- Gostei da separação de cada bloco no codigo da pagina index.html
- Separou os arquivos css de forma bem legal mesmo as variaveis podendo estar em um arquivo separado.
- Gostei muito de colocar variáveis
- Ficou bem proximo do template que te mandei
- Usou a classe modificadora da forma correta. Exemplo: as classes .active.
- Gostei de ter usado a TAG "link rel='preconnect'" do Google Fonts.
Dicas:
- Usar alguma metodologia para CSS, uma das mais utilizadas hoje globalmente é a metodologia BEM.
- Pegar mais prática com flexbox e estudar um pouco de Grid layout
- Usar mais classes para elementos, mesmo o pai ja tendo uma classe(Boas práticas)