Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PR de Correção #9

Draft
wants to merge 22 commits into
base: branch-para-correcao
Choose a base branch
from
Draft

PR de Correção #9

wants to merge 22 commits into from

Conversation

joaogolias
Copy link

POR FAVOR, EDITE ESSA MENSAGEM INSERINDO AS INFORMAÇÕES DO SEU PROJETO. COMECE APAGANDO ESSA LINHA.

INSIRA O NOME NOME DO PROJETO AQUI

O que funciona

  • DESCREVA O QUE FUNCIONA NO SEU PROJETO

O que não funciona

  • DESCREVA O QUE NÃO FUNCIONA NO SEU PROJETO

Link Surge

A PARTIR DA SEMANA 5, INSIRA AQUI O LINK DO SURGE. ANTES DISSO, APAGUE ESSAS DUAS LINHAS.

Imagens

TIRE PRINTS DAS TELAS DO SEU SITE E COLE AQUI

milenetaborda and others added 22 commits April 7, 2020 10:22
Bumps [acorn](https://github.com/acornjs/acorn) from 5.7.3 to 5.7.4.
- [Release notes](https://github.com/acornjs/acorn/releases)
- [Commits](acornjs/acorn@5.7.3...5.7.4)

Signed-off-by: dependabot[bot] <support@github.com>
Arquivos que alteramos em grupo hoje a tarde.
Um pouco mais de estilização RS
eita esqueci de criar uma branch nova. só commitando nem dei push
…-ninjas/acorn-5.7.4

Bump acorn from 5.7.3 to 5.7.4 in /future-ninjas
Copy link
Author

@joaogolias joaogolias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Lembrem de retirar os arquivos que o React cria automaticamente e vocês não usam

  2. Coloquem nomes melhores para as variáveis e as funções. O nome teste não diz o que a função faz, por exemplo.

  3. Retirem os console.log antes de subir a versão final dos projetos. Normalmente, não queremos que os usuários consigam ver informações no dev tools ao utilizarem o nosso site. Além disso, entendo que foi uma brincadeira, mas coloquem mensagens de erros mais decentes ("Erroooouuuwwww" não é muito legal)

Por fim, achei muito bom vocês terem utilizado media query para cuidar da responsividade. Fiquei muito contente de vocês terem utilizado transições no CSS de vocês. Além disso, o JS está muito bem escrito e fácil de entender. Meus 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.

5 participants