search

claudiocsx / one_page-projec_grupo_telegram

1 stars 0 forks source link

MOTIVATIONS SECTION #8

Open tvgc opened 2 years ago

VandersonT commented 2 years ago

@tvgc Finalizei a criação das duas mini-seções, languages e get-up, que fazem parte de 'Motivations' já com a responsividade. Também Adicionei uma animação na "imagem" do editor de código que fica na mini-seção languages, mas se quiser que remova é só falar.

tvgc commented 2 years ago

@VandersonT Parabéns pelo projeto! Ficou muito bom! Gostei demais das animações.

Gostaria de deixar dois pontos para melhoria: 1) Não é interessante o elemento "section" receber a classe reutilizável "container". Normalmente se separa outra classe reutilizável próprio para a section para criar os espaços entre as sessões. Como o nosso projeto não tinha um padrão de distância entre as sections, acabei deixando livre para cada um fazer. Portanto, o interessante é inserir a classe "container" no elemento filho que vem imediatamente ao elemento pai "section". Exemplo: -html:

<section class="section">
        <div class="container">
                ...
        </div>
  </section>

-css:

.section {
        padding-top: 50px;
        padding-down: 50px;
}
.container {
       max-width: 1440px;
       padding-inline: 16px;

2) O segundo ponto é não utilizar "margin-top" e "margin-bottom" mas sim paddings no elemento section, conforme está no exemplo acima.

VandersonT commented 2 years ago

Tudo certo agora, acabei de implementar a melhoria sugerida.