Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Filtro por localização do usuário #157

Open
wants to merge 20 commits into
base: develop
Choose a base branch
from

Conversation

kelvinsb
Copy link
Contributor

@kelvinsb kelvinsb commented May 15, 2024

Descrição

Para facilitar a vida de quem precisa saber os abrigos mais próximos, seja:

  • Alguém procurando abrigo
  • Procurando com itens de sobra para colaborar
  • Procurando com itens de falta para colaborar

Demonstração(desatualizado, olhar abaixo)

output.webm

Mudanças

  • Criado hook useGeolocation para facilitar o reuso do recurso da API
  • Adicionado/instalado @radix-ui/react-slider do shadcn
  • Criação do filtro de abrigos mais próximos
    • Criado slider para ser mais prático a seleção do raio
    • Botões com intervalos de 500m

Detalhes técnicos

  • Modificar modal de filtros para
    • Adicionar checkbox de filtro para abrigos próximos
    • (liberar caso o anterior seja selecionado) Input de raio(em metros)
  • Pegar coordenadas do usuário via API de geolocation do navegador
  • Enviar estes dados para o filtro do backend

Decisões

  • Foi deixado uns botões de soma e subtração baseado no intervalo(500m), e deixado na outra linha para não complicar a experiência do rangeSlider, principalmente no mobile que a tela já é pequena e complica se o slider for muito pequeno(já fica pequeno mesmo sem os botões nas laterais)
    • Estes foram criados para facilitar o uso, se por exemplo o usuário está com uma tela muito pequena e com dificuldade para acertar a distância os botões são para ajustar com o intervalo pequeno.
  • Foi utilizado um checkbox para habilitar estes campos porque no mobile eles por padrão vão utilizar muito espaço e deixa mais fácil de entender se tá utilizando ou não, se não aparece então qualquer um vai entender que não está sendo usado
  • Adicionado um texto com link de acordo com o dispositivo do usuário caso ele recuse ou tenha problemas com geolocalização, é usado apenas como auxilio já que podem ser vários problemas.

Backend

Atualização 18/05

Atualizar baseado no novo figma

image
image
image

implements #150
depends on SOS-RS/backend#93

Copy link

@AndersonCRocha AndersonCRocha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Favor analisar

@lucianomlima
Copy link
Member

@kelvinsb pode fazer um rebase e resolver os conflitos?

@kelvinsb
Copy link
Contributor Author

@kelvinsb pode fazer um rebase e resolver os conflitos?

@lucianomlima atualizei e resolvi os conflitos, se puder testar novamente e ver se tá tudo ok

@kelvinsb kelvinsb requested a review from elfiservice June 18, 2024 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants