search

millertime-homework / Emergency-Response

Emergency Response Game
0 stars 2 forks source link

Debug internet explorer #23

Open millertime-homework opened 11 years ago

millertime-homework commented 11 years ago

Lots of bugs have been resolved for internet explorer. #56 has been completed to warn users if their browser is IE. The current set of changes have been merged and will be released.

This ticket is staying open as development continues.

@moralismercatus is going to use Visual Studio to debug errors and provide points of failure.

millertime-homework commented 11 years ago

2 hours logged last night.

Found out why the 'emgames' page wasn't working, the tag for the external stylesheets had a 'media' attribute set to programmatically decide whether to load a normal or mobile stylesheet -- which apparently doesn't work in Internet Explorer. Also, I took out the hover events and replaced them with a click.

There continue to be issues with Internet Explorer <=8. Possibly related to trailing commas. Need to get Internet Explorer 8 running so we can figure out what exactly it's doing.

@moralismercatus proposed a great idea: dropping the IE7 requirement. Even on Windows XP, IE8 is not only available but the recommended version to use. If we get it working on IE8 and not IE7, we should discuss this -- with Pat too.

moralismercatus commented 11 years ago

1 hour logged.

Got XP running on a VM and ran ie7 - main menu a mess (purportedly fixed now) and unable to load game. Will try ie8 shortly.

millertime-homework commented 11 years ago

3 hours logged.

I got rid of all trailing commas. And trailing whitespace. And blank lines. And hard tabs.

Note: I might have missed some...

moralismercatus commented 11 years ago

1 hour logged.

For IE7 the main page works correctly, but the game does not start.

Since the bottom criteria has shifted to IE8, I downloaded that for XP.

The game works, but there are some problems that I encountered.

Note: I haven't rigorously tested. I haven't run through all scenarios in all ways.

1) Alpha doesn't seem to be supported. The glow around the doors is not feathered. It is all opaque. Then again, alpha is supported because Professor Bell is appearing by himself. This must mean IE8 doesn't support... feathering? I don't understand.

2) Once I finished Active Shooter, I went back to the main menu and attempted to play it again. Only a black screen and a non-rotating black spinner appear. Clicking on the page brings me back to the main menu. In fact, attempting to resume game or start the earthquake scenario has the same result. Restarting the browser allows for it to start again.

3) Not a big deal, but the corners are not rounded.

4) A single error appeared in the console "LOG: TypeError: 'triggers' is null or not an object' when you get killed by the active shooter. It doesn't seem to cause any problem.

millertime-homework commented 11 years ago

Wow! Nice work!

1) Post a screenshot of the glow around the doors. The "feathering" is part of the image itself. So what you're describing shouldn't be possible...

2) Very weird. Were you watching the console?

3) I'll look into that. It might not be supported.

4) Very useful. If you could copy and paste the message for that ending, I'd like to know specifically which ending it is, so we can trace the triggers/events causing the error you saw.

moralismercatus commented 11 years ago

1) glow

2) Yes. Nothing shows up.

4) Didn't happen when, in Active Shooter, I stayed in the room. Did happen when I left and got killed. Here's the message (BTW, I never took the elevator, I just went down the stairs): gameover

millertime-homework commented 11 years ago

Dude. That image is weird.

Go to this address and take a screenshot: http://web.clark.edu/emgames/Visual_Training/web/img/door.png

moralismercatus commented 11 years ago

Ok. Here it is. What was the point? glow2

millertime-homework commented 11 years ago

THAT is what it should look like in the game. We're not doing ANYTHING to change the way it's displayed, as far as I know. It should just be an image placed on the screen.

millertime-homework commented 11 years ago

Wait, that's not IE8. That's IE10. I meant IE8.

moralismercatus commented 11 years ago

Here's IE8:

glow3

Seems to work here. I don't know why it's not inside the game.

moralismercatus commented 11 years ago

This is going to take some work... Not tonight. I just tried the Earthquake scenario and after the earthquake triggered... nothing happened. It's like the after-earthquake scenario didn't get loaded. I'll debug with the console later.