riuri/crowds

Divisão

Closed this issue · 12 comments

italohdc commented

@riuri O que você está revisando agora? Para não estarmos trabalhando na mesma seção e dar erro de merge.

italohdc commented

Text Review:

  • Chapter 1
  • Chapter 2
  • Chapter 3
  • Chapter 4
  • Chapter 5
  • Chapter 6
  • Chapter 7

To-do (glitches found):

  • Fix yellow bar position on Chapter 1
  • Fix red button "Start Simulation" position on first slide, Chapter 2 (optional)
  • Nasa text too big on Chapter 3
  • Text misaligned on Chapter 6
riuri commented

Eu achei uma parte sem traduzir do trabalho voluntário e do experimental. Fora isso eu troquei detalhes e mexi nos textos dos círculos.

Tem dois textos que ficaram grandes, criei issue para eles

riuri commented

O yellow bar position eu arrumei diminuindo o tamanho do texto

riuri commented

Dê uma olhada na #6 e veja qual solução fica mais legal.

Por mim, terminada essa questão, podemos fechar tudo e fazer o pull request

italohdc commented

Eu já tinha editado o #6 . Dá um pull no seu repositório local e veja se aparece.

italohdc commented

@riuri A propósito, se você não usa nenhuma interface gráfica pro git, recomendo fortemente o GitKraken. Ele ajuda bastante a visualizar a árvore de commits haha

riuri commented

@italohdc já me recomendaram kkkkkkkk

Não o utilizo por não ser open source e por estar satisfeito com o gitg.

  • 😄1
italohdc commented
  • Fix red button "Start Simulation" position on first slide, Chapter 2 (optional)

Deixar esse assim mesmo?

riuri commented

Acho que sim, o lugar para mudar a posição está nos javascripts gerais

riuri commented

Podemos fazer o pull request?

italohdc commented

Então vamos fechar do jeito que tá!

Você acha que a gente deve colocar mais alguma informação em outro arquivo (créditos de tradução, informação de linguagem), ou só fazer o Pull Request desse jeito mesmo?

riuri commented

Desse jeito, seguimos todas as instruções do ncase