Skip to content

PR de Correção #2

Open
MirandaFayra wants to merge 2 commits into
correcao-projetofrom
master
Open

PR de Correção #2
MirandaFayra wants to merge 2 commits into
correcao-projetofrom
master

Conversation

@MirandaFayra
Copy link
Copy Markdown
Contributor

PR de correção, não margear!

@MirandaFayra
Copy link
Copy Markdown
Contributor Author

Seções
Parabéns time pelo projeto! As sessões ficaram bem dividas, contou uma história que facilitou o entendimento do usuário em relação ao projeto.

Na seção quero ser F4Bank, vocês colocaram um botão que não gera nenhuma ação para o usuário, seria interessante que esse botão levasse para um formulário de cadastro bem básico.

De resto, parabéns por como foram trabalhadas as seções.

Redes sociais
Todas as redes levavam o usuário para uma aba nova! Parabéns! A estilização dos ícones de redes sociais também ficou muito bom!

Teste mobile
Responsividade no inicio é bem desafiador mesmo! Alguns pontos que podemos melhorar

  • Muito interessante essa nav com o menu hambúrguer,talvez quando clicar expandir com as opções das seções tornaria o projeto mais interessante;
  • A barra de navegação móvel no mobile está cortando o texto, talvez não deixá-la estática na parte superior, seria uma melhor opção;
  • As letras e os textos estão se sobrepondo, seria interessante arrumar isso.

Estilização

Estilização ficou muito linda! Parabéns time! Amei a animação do logo meio voando 🤩 os textos ficaram muito bem elaborados e criativos, a imagem do cartão ficou sensacional, o site mostra que é um app para um público diverso (pauta super atual e necessária!)

Comentários Gerais Sobre o Projeto

Muito feliz em poder testar o projeto de vocês, deu para notar capricho e dedicação! Continuem assim! 🚀
Dediquem-se a aprimorar a responsividade, pois assim vocês já terão um projeto super legal para portfólio!

Ps: Não se esqueçam de margear as alterações do projeto em grupo antes do prazo de correção!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants