SOS-RS / frontend

Auxílio RS: Projetos de Resposta a Emergências por Chuvas e Alagamentos
https://sos-rs.com
MIT License
596 stars 328 forks source link

[BUG] Editar Itens do Abrigo não retorna ao abrigo #245

Open HbLuca opened 4 months ago

HbLuca commented 4 months ago

Descrição

Ao selecionar um abrigo na pagina inicial e seguir para a edição de itens, clicar no salvar e sair no "< Editar itens" no canto superior esquerdo, imagino que vá para a página do abrigo novamente.

Problema encontrado

Ao editar Itens do abrigo não retorna para a página do abrigo e sim para a página inicial

Prioridade

Solução proposta

Ao clicar no canto superior esquerdo "< Editar Itens" retornará para a página do abrigo.

Ambiente

Desktop Google Chrome - Windows 10 https://stg.sos-rs.com/

Evidência

https://www.loom.com/share/6c8327531d1543c290f617b0f3d6c1f2?sid=87eef38c-e6c7-4013-84d5-e16a47d7fed7

MayconPadilha commented 4 months ago

Continua com o bug? aqui repliquei e não esta voltando para a lista de abrigos, mas sim para o abrigo selecionado

HbLuca commented 4 months ago

@MayconPadilha no ambiente QA, segue. Ao invés de editar itens e retornar para a página do local, vai para home... https://www.loom.com/share/9e549bc05c7f43ea8a84b4a8b5708a49

MayconPadilha commented 4 months ago

@HbLuca não consegui replicar o problema: https://www.loom.com/share/c93c2270de8b42e7b153502af7223967?sid=4d31208e-89da-4ef4-9c25-eb5c8bbc0e87

HbLuca commented 4 months ago

@MayconPadilha Achei... Ao clicar no Editar itens ao invés de ser na parte da seta (<), leva-o para a home. No caso, acredito que ao clicar no "Editar Itens" ele fica conectado a seta (<) ou não? Caso sim, ao invés de seguir para a home, apenas retornar para a pagina do abrigo. https://www.loom.com/share/7f73bf11dcca48f4b2a9ce1881ab8daf

Ver a partir do 0:47

MayconPadilha commented 4 months ago

entendi o problema, vou tentar resolver amanha, esse fim de semana não tive muito tempo

HbLuca commented 4 months ago

@MayconPadilha Fechado, deixarei como responsável.

MayconPadilha commented 4 months ago

Realizei a pull request esta neste link: https://github.com/SOS-RS/frontend/pull/282

(Não sei como adicionava diretamente a task aqui)

HbLuca commented 4 months ago

@MayconPadilha Relaxe, adicionei aqui.

fagundesjg commented 4 months ago

https://github.com/SOS-RS/frontend/assets/29789354/b813b87c-85d3-4b78-b921-d90cd6542da7

Conforme vídeo acima, testado no ambiente de dev podemos ver que o comportamento está dentro do esperado que é:

  1. Ao clicar na seta de voltar ( < ) ele volta a página anterior
  2. Ao clicar no título do header, volta a página inicial. O voltar a página inicial ao clicar no título é um comportamento "padrão" encontrado em vários sites. E muitos usuários estavam tendo a navegação afetada quando desejavam voltar a página inicial de maneira simples.
HbLuca commented 4 months ago

Faltou realização de teste ao clicar em Editar Itens. (Clicou no <, apenas) O teste que realizou para voltar para a home, foi da parte do abrigo e não da edição de itens do abrigo.

Se realizar testes em páginas de banco, por exemplo, ao ir para a terceira página, ou abertura de um pop-up, o comportamento apontado é o adequado, de retornar para a segunda página e não para a página inicial.

Ex: Nubank Conta - Gastos Previstos - Contas pausadas e ocultadas.

No caso do SOS-RS ao clicar em Editar Itens, o correto seria ir para a página do abrigo e não para a página inicial, a não ser que esteja na pagina do abrigo. Em muitas páginas ao menos na versão desktop existe até o home que ai sim, retorna para a inicial.

Vale lembrar que deveria ser revisado pela equipe de @SOS-RS/designers sobre o fluxo ao clicar nos botões, no Figma. Edit: Segue local do figma que está presente o fluxo... https://www.figma.com/design/uy6ukD6inEdpvaKVhIyobX/SOS-RS?node-id=241-257 @SOS-RS/pos e @SOS-RS/designers favor observar.

teodorogit commented 4 months ago

Olá, quero contribuir para a resolução desse Bug

HbLuca commented 4 months ago

@teodorogit boa noite, favor verificar PR https://github.com/SOS-RS/frontend/pull/282 o @MayconPadilha está trabalhando nela, converse com ele.