Skip to content

Conversation

sombraSoft
Copy link
Contributor

Adds a button right next to the shelter address to open it in Google Maps
The InfoRow component was modified to accept children in order to render this link.

chrome_wbBFKfzGuA

@LucasJantschChitolina
Copy link

LucasJantschChitolina commented May 11, 2024

image

@sombraSoft, olhando o gif eu pensei que talvez seria interessante fazer um pequeno preview do link do Google Maps. Assim ficaria melhor ocupado o espaço do componente e também facilitaria pra pessoa clicar no link.

Seria importante mudar a opacidade do preview ao fazer um over e exibir um texto indicando que leva ao Google Maps.

O que acha? Foi uma ideia rápida que tive, mas se for muito trabalhosa a implementação pode relevar.

@henriquecfreitas
Copy link

@LucasJantschChitolina acho que a ideia é bem válida, porém eu não vincularia ela ao merge desse PR, que por si só já vai agilizar bastante a experiência dos usuários que precisem abrir o endereço do abrigo no maps

@sombraSoft
Copy link
Contributor Author

sombraSoft commented May 11, 2024

adicionar o mapa ali pode trazer legal mas tem que ver como fica na visão mobile também. como o @henriquecfreitas mencionou, o PR por si só já traz um valor.

@LucasJantschChitolina
Copy link

Entendi, então eu vou pensando na ideia por aqui e futuramente lanço um novo PR pra isso.

Copy link
Contributor

@kelvinsb kelvinsb left a comment

Choose a reason for hiding this comment

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

Ficou bom e simples!

@antoniolucasnobar
Copy link

@sombraSoft não seria mais preciso utilizar a latitude e longitude ao invés do endereço?

@sombraSoft
Copy link
Contributor Author

@antoniolucasnobar provavelmente, mas quando o usuário abre o maps ele não vai traduzir para um endereço de volta quando ele abre. exemplo:
usando lat/lon
Screenshot_20240512-223455~3.png

usando endereço
Screenshot_20240512-223554~2.png

acho que na prática dá no mesmo, mas usando lat/lon ele não mostra o nome do lugar destino, contato, fotos etc.

Copy link

@burgos01 burgos01 left a comment

Choose a reason for hiding this comment

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

Da forma atual já agilizaria bastante a usabilidade do sistema

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.