search

humitos / osm-pois

Show POIs from OSM in a map using OverpassAPI
http://upoi.org/
GNU General Public License v2.0
22 stars 21 forks source link

fix html #66

Closed nico-stefani closed 4 years ago

51114u9 commented 8 years ago

@nicolas471 Si bien este parche soluciona el problema también modifica otras cosas que podría no gustar a todos.

Yo creo que deberías crear un parche con sólo el cambio para corregir la cabecera.

El resto (bibliotecas offline, validacion html, simplificacion estilo) deberían ir en otros PR. Siempre atomizar los cambios para que sean entendibles.

nico-stefani commented 8 years ago

@51114u9 gracias por la aclaracion, lo voy a corregir!

humitos commented 8 years ago

@nicolas471 gracias por el PR.

Como bien dice @51114u9 es importante que mantegamos los PR limpios y concretos (atómicos); así a la hora de evaluarlo uno puede estar concentrado en un cambio específico sin tener que perderse en muchas cosas que probablemente no estén relacionadas.

Además, puede ser que 4 de los 5 cambios propuestos sean buenísimos y si están mezclados no hay forma de aceptar los 4 que están buenos.