SOS-RS / frontend

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

[SUGESTÃO] Adição de Botão "Voltar ao Topo" #215

Open HbLuca opened 6 months ago

HbLuca commented 6 months ago

Descrição Verificar a possibilidade de adição de botão, na parte inferior direita da página, após "scrollar", com a equipe de Design/UX e DEV.

Motivo Se a pessoa não realiza a pesquisa por filtro, mas apenas for scrollar/rolar a página, para que a mesma não atualize-a e sim clique no botão para retornar ao início.

Ambiente Desktop (Google Chrome) - Windows 10

Ideia referente a sugestão bt_vt

HbLuca commented 6 months ago

Correlacionado: 63 131 133

kevinDsousa commented 6 months ago

Verificando esta atividade

HbLuca commented 6 months ago

@kevinDsousa Caso necessário, verificar com a equipe de Design/UX, sobre...

Lmedeiros-leiman commented 6 months ago

criei uma implementação básica para essa sugestão utilizando as cores principais do cabeçário.

https://github.com/SOS-RS/frontend/pull/280

barbiebrega commented 6 months ago

Fiz o design do botão aqui

O primeiro é no estado normal e o segundo no hover. Sugiro usar uma margem de 24px por 24px no desktop. No mobile acho que podemos esconder o botão.

Qualquer dúvida só chamar.

HbLuca commented 5 months ago

@Lmedeiros-leiman favor observar o que a @barbiebrega digitou...

Lmedeiros-leiman commented 5 months ago

OK, implementado o Design.

LeandroOliveiraDeQueiroz commented 5 months ago

Gostaria de contribuir com essa task. Pode assinar para mim?

Lmedeiros-leiman commented 5 months ago

eu já implementei a solução, ela não foi visualizada. https://github.com/SOS-RS/frontend/pull/280

LeandroOliveiraDeQueiroz commented 5 months ago

eu já implementei a solução, ela não foi visualizada. #280

Dei uma olhada no código e vi que você fez alteração na sua implementação seguindo as instruções da @barbiebrega. Sem problemas, vou deletar aquilo que eu programei. Eu senti que o fluxo da task ficou confuso: ela estava em Disponível para dev, você já tinha feito e não estava assinada a ninguém.

larissapissurno commented 5 months ago

precisamos de mais um approval cc @SOS-RS/frontenders

larissapissurno commented 5 months ago

PR mergeado em develop. O GH está fechando automaticamente as issues atreladas ao PR mergeado, estou reabrindo pra podermos mandar essa alteração pra staging e depois mover pra coluna de teste

larissapissurno commented 5 months ago

PR pra deploy em staging: https://github.com/SOS-RS/frontend/pull/311

@fagundesjg @SOS-RS/frontenders poderiam revisar 🙏 ?

larissapissurno commented 5 months ago

ps: PR testado em https://dev.sos-rs.com/ e atende os requisitos

HbLuca commented 5 months ago

@Lmedeiros-leiman @barbiebrega @larissapissurno Temos a possibilidade disso ser implementado também em versão mobile? Realizei o teste a pouco e fica o questionamento...

https://www.loom.com/share/49c2c398bec8441780ac48955a009ba7?sid=1f5db4e1-18ea-411b-a321-a843d7de9dd5

larissapissurno commented 5 months ago

@Lmedeiros-leiman @barbiebrega @larissapissurno Temos a possibilidade disso ser implementado também em versão mobile? Realizei o teste a pouco e fica o questionamento...

https://www.loom.com/share/49c2c398bec8441780ac48955a009ba7?sid=1f5db4e1-18ea-411b-a321-a843d7de9dd5

acredito que sim @HbLuca , podemos fazer um ajuste ali nas classes de responsividade, o que acha @Lmedeiros-leiman ?

Lmedeiros-leiman commented 5 months ago

seria apenas necessário o design para a versão mobile.

barbiebrega commented 5 months ago

Oi pessoal. Eu orientei esconder no mobile esse botão por dois motivos: 1 - como a página é pequena, possivelmente o botão vai esconder conteúdos importantes 2 - no mobile a rolagem com os dedos é muito mais rápida do que no desktop, tornando o botão desnecessário ao meu ver

HbLuca commented 5 months ago

@barbiebrega faz sentido os 2. Caso nao adicione a parte referente ao botao, seguirei com os testes no desktop.

HbLuca commented 5 months ago

Com relacao a versao desktop, esta de acordo, tanto em DEV, quanto QA. No aguardo referente a versao mobile para prosseguir com os testes ou/finaliza-lo. @rhuam @giggio @larissapissurno

Vide Loom: https://www.loom.com/share/994fe549b0eb4ace9140c7cda16d2e55?sid=7854e145-8b5a-4e7b-8324-b5b5c3cf138aari