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

Fix index layout #81

Closed
wants to merge 2 commits into from
Closed

Fix index layout #81

wants to merge 2 commits into from

Conversation

javiayala
Copy link
Contributor

Hice muchos cambios en CSS para el layout del Index. Quedo mas nice.
Inclui tambien un box de info on Hover del Report solo que tiene la información estatica. Eso lo hare en otro branch para mantener en este Pull request puro cambio de layout y nada que tenga que ver con migraciones para hacer que guarde la direccion que da Google.

@javiayala
Copy link
Contributor Author

@wilhelmbot Checa este pull request y dale merge. Necesito hacer otra branch pero ya con estos cambios en master para agregar la funcionalidad del la caja con informacion en el Index

@defvol
Copy link
Member

defvol commented Feb 26, 2013

@jawks estuve desconectado el fin

cuando quieras hacer un branch que dependa de esto, abrelo. Es decir haz un branch a partir de este branch no de master, y cuando termines puedes hacer un pull request del 2do branch a este branch.

o la otra opción es que lo revise alguien más.

En lo personal yo creo que si son fixes de estilo deben entrar rápido a master.

@defvol
Copy link
Member

defvol commented Feb 26, 2013

En general está bueno este pull request, pero creo que se deberían atender las siguientes observaciones antes de entrar a master:

  1. La caja se queda estática, y si muevo el mapa para alejarme de un pin, ésta sigue presente a pesar de que ya no estoy en el pin. Creo que el comportamiento esperado es que al hacer click en el pin se vea la caja, y que cuando se haga click en otro lado se cierre, no?

Screen Shot 2013-02-26 at 9 26 57 AM

  1. Desapareció el control de zoom del mapa. Creo que esto es importante para la usabilidad no? Yo metí el control hace unos commits atrás.
  2. Hay que agregar el link en VER MÁS

qué opinan?

@kurenn
Copy link
Member

kurenn commented Apr 23, 2013

Creo que si le hace falta mucho a este pull request....y pienso que seria mejor utilizar los info windows nativos de google maps
https://developers.google.com/maps/documentation/javascript/examples/infowindow-simple

Tambien se pueden customizar, https://groups.google.com/forum/?fromgroups=#!topic/google-maps-js-api-v3/TJmmtVhWmng por que lo siento medio hackish tambien esto.....

@defvol
Copy link
Member

defvol commented Apr 23, 2013

Cerramos esto entonces y en el #65 lo atendemos

@defvol defvol closed this Apr 23, 2013
@defvol defvol deleted the fix-index-layout branch April 23, 2013 21:06
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.

3 participants