Skip to content

Comments

Correção PR#15

Open
MirandaFayra wants to merge 32 commits intocorrecao-projetofrom
master
Open

Correção PR#15
MirandaFayra wants to merge 32 commits intocorrecao-projetofrom
master

Conversation

@MirandaFayra
Copy link

Correção PR não margear

gabazevdo and others added 30 commits June 26, 2021 11:54
Iniciando o projeto - inserindo os arquivos base
Formatando o readme.md: deixando mais apresentável
Inserindo icones de redesSociais - menu e sections
Inserindo imagens e modificando (bem pouquinho) no CSS
Realizei pequenas modificações no html e no css
@MirandaFayra
Copy link
Author

Seções
As seções estão bem organizadas, eu senti uma harmonia entre as imagens selecionadas e
as seções. Cumpriu o prometido: informações e imagens estão bem coerentes. O efeito de
ancora ficou bem útil e facilitou a navegação.

Redes sociais
Consegui localizar os ícones facilmente e fui direcionado para cada rede social
correspondente com o ícone.

Teste mobile
Todas as seções/funcionalidade estão coerentes com as do site no modo desktop, porém
não consegui ler todas informações por que tem uns títulos/palavras cortadas.

Estilização
Achei a estilização muito boa, me senti em um site real, a harmonização entre imagens
deu um toque especial. Para colaborar eu daria uma atenção maior a cor usada nos ícones das
redes sociais, pois eu acho que eles podem ser mais percebidos quando usamos mais contrates, nesse caso
o branco cairia bem. Na primeira seção tem um quadrado preto do lado direito com o nome
"upfour" eu acho que ele deu uma quebrada no layout da página, poderia colocar o mesmo fundo
e realinhar esse título.

Comentários Gerais Sobre o Projeto
O projeto muito bom! É semelhante a um site real, oferece um experiência visual agradável, seções
com conteúdos coerentes com a imagem, navegação por ancoras, links para redes sociais funcionando como esperado
e funciona parcialmente bem em celular, precisando fazer ajustes nos textos para que consigamos ler.
Eu visitaria esse site de novo ♥

Parabéns!✨

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.

4 participants