search

codeforamerica / adopt-a-hydrant

A web application that allows citizens to "adopt" civic infrastructure, such as fire hydrants that need to be shoveled out after it snows.
http://adopt-a-hydrant.herokuapp.com/
BSD 3-Clause "New" or "Revised" License
713 stars 233 forks source link

Application does not work in IE7 #18

Open zzolo opened 12 years ago

zzolo commented 12 years ago

Specifically the map does not work. I was not able to debug fully yet, but wanted to get this issue in. There were a number of errors that popped up.

I don't think it is an issue with Google Maps as it should support IE7 still.

More, real information coming soon.

sferik commented 12 years ago

Thanks for the bug report, Alan. I don't have access to Windows VM at the moment (which explains why it's not working in the first place) but I'd love a detailed bug report or even a patch if you think you know how to fix it.

I'll be at my folks place over Thanksgiving, which is a good time for me to debug Windows issues. I'll make sure to take a look when I'm home. :wink:

sunnyrjuneja commented 11 years ago

I was able to confirm this bug exists. I tried to see if I could find a relevant stack trace but there were no errors in the console. I was able to replicate it by opening IE 10 in IE 7 quirks mode.

Is there anything else I can do?

mapmeld commented 11 years ago

Is there an online version of the site with this problem? Testing on browserstack.com, I was able to open adopt-a-hydrant.herokuapp.com but the map was too wide for the page (it appears further down on the page, below the left sidebar).

If this is an older version of the code, the two main IE JS bugs that I look for are: console.log (which isn't supported unless you're debugging, so anyone debugging the site will see no problems) and a comma at the end of JSON or JS objects (syntax error in IE)

sunnyrjuneja commented 11 years ago

Actually, mapmeld, it looks like browser stack is replicating the problem exactly. I think zzolo and I were both experiencing the same issue you described. I did not realize if you scrolled down the map was there. It looks like this is a layout issue in IE7.