Skip to content
This repository has been archived by the owner on Mar 20, 2018. It is now read-only.

Informações pessoais (João Henrique) e #14. #31

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Informações pessoais (João Henrique) e #14. #31

wants to merge 2 commits into from

Conversation

jhkersul
Copy link

@deborasetton deborasetton changed the title Issue 1 & 14 Informações pessoais (João Henrique) e #14. Mar 21, 2014
@ghost
Copy link

ghost commented Mar 22, 2014

Comentários

Alguns comentários gerais sobre esse pull request:

  1. O código ficou ótimo, parabéns!

  2. As mensagens de commit também ficaram muito boas: em Inglês, e explicando o que foi feito.

  3. O ideal, para melhor organização do que acontece no repositório, é que exista um pull request para cada issue. No entanto, este pull request está resolvendo duas issues ao mesmo tempo: a [Everybody] Add personal information #1 e a Close modal when user clicks on the semi-transparent black background. #14.

  4. Existem algumas pequenas melhorias que podem ser feitas (sempre existem!). Veja os comentários aqui: https://github.com/infosimples/meet-the-class/pull/31/files


Alterações pendentes

Dados os comentários acima, as nossas sugestões de alteração estão abaixo. Faça as alterações nessa ordem; caso contrário, podem acontecer coisas inesperadas.

  1. Alterar os arquivos para consertar os pontos levantados no comentário 4.

  2. Fazer um novo commit.

  3. Finalmente, fazer um push e verificar se as alterações foram enviadas para o GitHub.

Observação 1: Nós não vamos separar os commits em dois pull requests, apenas para facilitar. Mas isso poderia ser feito. Se você quiser saber como, fale conosco!

Observação 2: Se você tiver alguma dúvida sobre como fazer commits, pushs, etc, dê uma olhada nos slides, ou fale conosco.

@@ -177,6 +177,11 @@ header {
z-index: 3;
}

.externalarea {
Copy link

Choose a reason for hiding this comment

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

Nós costumamos separar as palavras com um hífen (veja os exemplos no restante do arquivo). Nesse caso, o nome dessa classe seria external-area.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant