midudev / covid-vacuna

App para ver el estado de la vacunación COVID-19 en España
https://covid-vacuna.app
Apache License 2.0
365 stars 126 forks source link

Añadir islas canarias al mapa #98

Closed sergioedo closed 3 years ago

sergioedo commented 3 years ago

He añadido los datos de las islas canarias al mapa, utilizando una proyección específica para este caso (traer las islas más cerquita de la peninsula). La idea la saque de este repo (donde además procesan los datos públicos para generarlos en formato TopoJSON):

https://github.com/martgnz/es-atlas

La "caja" para enmarcar las islas puede hacerse abierta (por defecto) o cerrada (por si gusta más, via props). Le puse un color gris para que se vea bien en dark/light mode. Ajusté un poco el viewbox del SVG para que quedara más centrado, al haber más datos en el mapa.

vercel[bot] commented 3 years ago

@sergioedo is attempting to deploy a commit to a Personal Account owned by @midudev on Vercel.

@midudev first needs to authorize it.

vercel[bot] commented 3 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/midudev/covid-vacuna/453175p9d
✅ Preview: https://covid-vacuna-git-fork-sergioedo-feature-add-canary-islands-map.midudev.vercel.app

sergioedo commented 3 years ago

He movido la capa del contenedor al fondo (si no tapaba parte de las islas y no podías hacer el overlay sobre ellas)

jelowin commented 3 years ago

Hola @sergioedo , lo veo guay buen trabajo :). La única pega que le veo es que sale el mismo tooltip para todas las islas y para Fuerteventura y Lanzarote directamente no sale tooltip. No sé si es posible añadir el nombre y los datos de cada isla en el tooltip? Sino es así, alomejor simplemente podríamos dejar un único tooltip para todo el cuadro de Canarias.

¿Que opinas?