search

Honricke / Servidor-PBHI

0 stars 1 forks source link

Problemas encontrados durante o rebase e merge com o repositório antigo #1

Open FlyingWolFox opened 1 year ago

FlyingWolFox commented 1 year ago

Resolvidos:

Critico:

Grave:

Leve:

Informativo:

Crítico

Problemas que quebram o site ou o jogo sem possível solução pelo usuário, ou que afetam grandemente o desenvolvimento e operação do site.

~C1. O botão about está com o id desatualizado~

~O botão tenta achar modalInfo, mas o id foi mudado para modal_info~

~C2. Fluxograma enviando string errada para o servidor~ Movido de crítico para leve

~No jogo do grupos fluxograma, quando se há pouco espaço de tela, os nós "Colocar na esquerda/direita" se reduzem para "Esquerda/Direita". O jogo envia pro servidor a string do nó, só que o banco de dados não lida com esses nomes reduzidos. Isso deve ser corrigido no jogo (preferível), no servidor ou no banco de dados~

~C3. README de como rodar o servidor~

~Colocar instruções no README.md de como rodar o servidor~

~C4. O jogo dominó da diferença não gera peças ao entrar~

~Exemplo:~ Captura de Tela (24)

~C5. O jogo da repetição está com o drag n drop bloqueado(fases 1 e 2)~

~Na fase 1 o drag n drop fica bloqueado depois de arrastar a peça pela primeira vez, o container de peças fica como estando ocupado. Na fase 2 o container sequência fica como ocupado desde o ínicio, impossibilitando o jogador de arrastar as peças.~

C6. O jogo do fluxograma está com o CSS quebrado

Exemplo: Captura de Tela (23)

~C7. O jogo criar sequência não gera peças ao entrar~

~Similar ao dominó da diferença~

C8. O modo tela cheia está desativado nos jogos logix e fluxograma

o botão de tela cheia não funciona em ambos os jogos

~C9. Todos os jogos estão disponíveis para todos os anos~

~Um erro na verificação do ano. Na seleção de ano, os anos são armazenado em caixa alta (ex. "PRIMEIRO ANO"), mas os if checam com primeira maiúscula (ex. "Primeiro Ano"):~

image

image

~(As imagems são do mesmo arquivo, public_html/selecao/index.html)~

Grave

Problemas que que quebram o site ou o jogo, mas que podem ser contornados pelo usuário.

~G1. Botão "fechar" quebrado~

~O botão "fechar" está quebrado em todos jogos, voltando para o index.html do dir anterior (..) que é selecao/jogos/ ao invés de selecao/ (já que o dir de jogos não tem um index).~

G2. Estilos para mobile não estão sendo aplicados

O estilos para mobile não estão sendo aplicados. As referências ao mobile-style.css foram removidas

~G3. Vários jogos não chamam cancelarAudioModal()~

~Em vário jogos, o botão "Entendi" (do modal info) não chama cancelarAudioModal(), assim se o usuário apertar para ouvir o texto e pressionar o botão "Entendi", o audio não vai ser interrompido~

G4. Modal Info está com o estilo quebrado

O modal aberto pelo botão info ("?") está parcialmente estilizada. Por exemplo, o texto fica maior que o modal, ficando fora das bordas

~G5. No Jogo da Repetição, o botão de falar do modal info não chama ativarAudioModal()~

~O botão com id="botao_de_voz" não possui onclick="ativarAudioModal()". Bug introduzido devido ao erro na merge com as alterações de responsividade, sendo esquecido de transformar~

<button id="botao_de_voz">
    <span id="texto_de_voz">
        <img src="../img/svg/audio.svg" alt="botão de aúdio" id="img_de_voz">
    </span>
</button>

~em~

<button id="botao_de_voz" onclick="ativarAudioModal()">Falar!</button>

~O que resultou em perda de código relacionado à audio~

~G6. Grid do game_section possui regra com resultados diferentes no chrome vs firefox~

section#game_section, que é um grid, possui a propriedade grid-template-rows: 0% calc(100% - 100px) (regra main#main_body section#game_section, page-style-games.css, linha 240). Só que essa regra possui diferentes efeitos dependendo do navegador, especificamnte o 0%.

Chrome:

image

Firefox:

image

(Perceba o quadrado amarelo/laranja mostrando a margem do elemento)

Alterar grid-template-rows: 0% calc(100% - 100px); para grid-template-rows: fit-content(0) calc(100% - 100px); faz a regra ter o mesmo efeito nos dois navegadores. fit-content(0) faz o header ter o menor tamanho possível, mas não faz o item ter tamanho 0, então o segundo item é renderizado abaixo do header. Assim também é necessário retirar o margin-top: 100px da regra main#main_body section#game_section main#game_content (page-style-games.css, linha 301) para completar a correção. Essa correção parece não causar quase nenhuma alteração de layout no chrome e corrige no firefox.

Depois da mudança:

Chrome:

image

Firefox

image

Correção sugerida:

diff --git a/public_html/style/page-style-games.css b/public_html/style/page-style-games.css
index 68bfa7cc..ddfa74e8 100644
--- a/public_html/style/page-style-games.css
+++ b/public_html/style/page-style-games.css
@@ -181,7 +181,7 @@ main#main_body section#game_section {
   width: 100%;
   min-height: calc(100vh - 85px - 75px);
   display: grid;
-  grid-template-rows: 0% calc(100% - 100px);
+  grid-template-rows: fit-content(0) calc(100% - 100px);
   border-radius: 10px;
   background-color: #fff;
   box-shadow: 2px 2px 4px rgba(0, 0, 0, 0.25);
@@ -306,7 +306,6 @@ main#main_body section#game_section main#game_content {
   flex-direction: column;
   align-items: center;
   justify-content: space-between;
-  margin-top: 100px;
 }
 main#main_body section#game_section main#game_content div#box_central {
   width: 100%;
diff --git a/public_html/style/page-style-games.scss b/public_html/style/page-style-games.scss
index 2b752648..1a052b19 100644
--- a/public_html/style/page-style-games.scss
+++ b/public_html/style/page-style-games.scss
@@ -182,7 +182,7 @@ main#main_body{
         min-height: calc(100vh - 85px - 75px);

         display: grid;
-        grid-template-rows: 0% calc(100% - 100px);
+        grid-template-rows: fit-content(0) calc(100% - 100px);

         border-radius: 10px;
         background-color: #fff;
@@ -350,8 +350,6 @@ main#main_body{
             align-items: center;
             justify-content: space-between;

-            margin-top: 100px;
-
             div#box_central{
                 width: 100%;
                 height: 100%;

Jogos afetados:

*Logics é afetado diferente, firefox e chrome possuem o mesmo comportamento (primeiro item possui tamanho 0), então não está quebrado, mas a correção também se aplica a esse jogo e não causa alterações no layout

Fluxograma é o único que não afetado, mas isso é porque esse jogo não sofreu a alteração do css para o modelo page-style-games.css, então usa um estilo próprio (grupos fluxograma/game-style.css)

Leve

Problemas que não quebram o site ou o jogo, mas que precisam ser resolvidos

~L1. Mudar o nome dos anos para caixa alta~

~Lincoln queria que o nome dos anos aparecesse em caixa alta nas páginas de seleção (no topo da página). Isso foi consertado por Davi alterando as strings que são colocadas no Local Storage, porém isso quebra alguns jogos (e possíveis outras páginas) já que usam essa string em um switch-case.~

~A solução seria alterar as strings para caixa alta somente nas páginas dos jogos, usando o método toUpperCase() ou toLocaleUpperCase(). O local para alterar está na linha com "MudandoAno.innerHTML"~

L2. Peças pequenas e grandes não são diferenciáveis (possuem o mesmo tamanho)

Jogos afetados:

L3. IDs de containers (sequência/núcleo) estão confusos

Os ids "container-sequencia" e "container-nucleo" estão invertidos em alguns jogos. Jogos afetados:

No geral, é questionável usar "container-nucleo" onde o jogador arrasta as peças, já que é específico a um jogo só. Jogos afetados:

~L4. Remover código onclick="resetaTimeStamps()" dos botões de "Terminei"~

~É necessário remover o código onclick="resetaTimeStamps()" dos botões de "Terminei" de todos jogos (a função não existe mais)~

~Jogos afetados:~

L5. CSS de materiaisProfessores.html precisa ser ajustado

O CSS de materiaisProfessores.html precisa ser ajustado. Os botões e a fonte não se reajusta para displays pequenos

~L6. Jogo dos Grupos: Alinhamento incorreto da restrição da direita~

~A restrição da direita está alinhada inferior a da esquerda, mesmo quando não há interseção. Era para estar na mesma altura que a restrição da caixa da esquerda~

L7. O texto que é falado no Grupos Fluxograma está errado

O texto que é falado no Grupos Fluxograma está errado. O texto corresponde ao Jogo do Completar

L8. Cor do texto do Dominó está errada

O texto de "DOMINÓ" (em cima da caixa) é para ser amarelo ou azul? (ser da classe titulo-sequencia ou titulo-nucleo? alterar essas regras/omitir uma? o texto azul tem regras extras de margens!)

L9. Reticências no texto do modal do Dominó

No texto sobre (mostrado pelo botão "?"), há reticencias no meio do texto. Isso é intencional?

L10. Jogo dos Grupos: Peças aumentarem de tamanho quando arrastadas parece quebrado

A função de aumentar o tamanho das peças quando arrastadas parece não funcionar corretamente. Ou não se aplica a todas as peças ou certas peças possuem um aumento de tamanho muito grande. Tem que investigar mais ainda

L11 Grupo Fluxograma: index.html não tem tag de fechamento do body (</body>)

~L12. Sequência Decrescente: Título da página está incompleto~

~Sequência Decrescente: Título da página só mostra "Jogo da Sequência", que é igual ao do jogo da sequencia~

L13. A imagem capa do Sequência Decrescente possui uma sombra diferente das outras

Dark Reader ajuda facilita ver isso

L14. Mudanças não presentes no modelo

Não estão no modelo (SCSS):

L15. Títulos de algumas páginas estão com erros

Algumas páginas do ínicio (não dos jogos) possuem títulos errados ou incompletos.

Títulos que possuem esse problema são as seguintes:

Caminho Título Descrição
index.html Tem Lógica Seleção de Ano
selecao/index.html Tem Lógica - Seleção de Ano Seleção de Jogo
materiaisProfessores.html Tem Lógica - Materiais Professores Materiais para Professores
professores.html Tem Lógica - Professores Acesso Professores
selecao/selecionarFase.html Tem Lógica - Seleção do Ano Seleção de Fase

Talvez seja desejável títulos que sejam parecidos com a descrição

L16. Fluxograma enviando string errada para o servidor

Movi esse erro de crítico para leve porque no momento os dados que são coletados através dessa string não estão sendo utilizados nos relatórios, portanto a prioridade disso é baixa No jogo do grupos fluxograma, quando se há pouco espaço de tela, os nós "Colocar na esquerda/direita" se reduzem para "Esquerda/Direita". O jogo envia pro servidor a string do nó, só que o banco de dados não lida com esses nomes reduzidos. Isso deve ser corrigido no jogo (preferível), no servidor ou no banco de dados

Informativo

Problemas que não afetam funcionamento ou que são opcionais de resolver, mas que devem ser considerados

I1. Modelo talvez precise ser atualizado

Ele foi feito baseado em html e css antigo. Talvez seja necessário atualizar o modelo para refletir as mudanças feitas no site

I2. Os documentos da seção de Materiais Para Professores está em selecao/jogos/documentos/

Alteração feita na reorganização do repo em 3e18c2f. Isso deveria estar na raiz, não?

I3. O documento documento.pdf (public_html/selecao/jogos/documentos/documento.pdf) é a junção dos outros três documentos presentes no diretório. Ele é necessário?

I4. ~João Vittor Vale a pena alterar o número de fases baseado no ano? getFasesPorAno() existe, mas não é usada. Jayme falou que você ia pesquisar sobre isso, se vale a pena~

I5. Arquivos não usados

Arquivos não usados que parecem ser extras ou versões antigas de outros arquivos:

Os index.html tem o mesmo problema do header só ter um botão para páginas dos professores

I6. Atualizar lockfile

É preciso atualizar a versão do lockfile package-lock.json (nodejs)

I7. Nomes de Ano devem ser padronizados

Nomes de Ano devem ser padronizados: Só a primeira letra maiúscula ou tudo maiúscula

I9. Botão de Desligar audio lança exceção

O botão chama botaoGlobalDeVozAtivado(this) que não existe

FlyingWolFox commented 1 year ago

Problema G5 adicionado

FlyingWolFox commented 1 year ago

Problema L15 adicionado

FlyingWolFox commented 1 year ago

Problema G6 adicionado