Repository navigation
Feature/peca fantasma - #2
Merged
Merged
Conversation
3 tasks done
ZeCarlos18
approved these changes
Oct 11, 2026
ZeCarlos18
left a comment
Collaborator
There was a problem hiding this comment.
Testei a branch e a peça fantasma está funcionando. Aprovado. Deixo abaixo algumas sugestões que não bloqueiam o merge.
O que testei
- Sombra: aparece na posição de encaixe da peça e acompanha os movimentos para os lados e a rotação.
- Pilha: com peças empilhadas, a sombra para em cima da pilha, sempre numa posição livre.
- Tecla G: liga e desliga a sombra durante a partida. Na pausa não faz nada, o que parece intencional.
- Teclas existentes: Espaço, ESC e setas continuam funcionando normalmente.
- Telas pequenas: a tela inicial com os 7 controles cabe sem cortar em 300px e 450px.
- Console: nenhum erro.
Sugestões (não bloqueiam)
- Título do PR: que tal
feat: adiciona peça fantasma, no padrão Conventional Commits que usamos no PR #1? Os commits já estão nesse padrão. - Espaço sobrando no fim da linha: em
function landingY() {efunction drawGhost() {, noinde.html. - Alinhamento em
CONTROLS: a linha{ keys: ['G'], action: 'ligar / desligar sombra' }ficou desalinhada das outras. É só estético.
Aviso: conflito com o PR #3 (cronômetro)
Este PR e o #3 alteram os mesmos pontos do inde.html. Isso é proposital, porque a atividade pede que a equipe passe por um conflito de PR. Quem entrar na main por último vai ter conflito em 2 trechos:
- Logo depois de
lose():var ghost/toggleGhost()× seção do cronômetro. Aqui é só manter os dois lados. - Logo depois de
drawRows():drawGhost()×drawTimer(). Cuidado neste: o Git junta o}final das duas funções, então "aceitar os dois lados" direto no editor gera erro de sintaxe. Ao resolver, é preciso fechar odrawTimer()com}antes dodrawGhost().
Se este PR entrar primeiro, eu resolvo o conflito na branch do cronômetro mantendo a tecla G e a sombra.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adiciona a peça fantasma: uma sombra semitransparente que mostra onde a peça atual vai cair.
O que foi feito:
Como testar: