-
Notifications
You must be signed in to change notification settings - Fork 6
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
[IE11] Splash appears in the top left of the screen at startup, then centers #408
Comments
Could it be related to the CSS in #407 ? |
#407 was attributed to a minor image dimension change. We can see if an image size problem is causing this issue as well (say, if the image has size 0,0) at the beginning. |
I can still see the problem in my virtual box IE11 for http://www.colorado.edu/physics/phet/dev/html/energy-skate-park-basics/1.3.1-dev.3/energy-skate-park-basics_en.html However, when testing with a built faradays law using the fix from #407 I no longer see the problem. |
Well it is still a problem in that version, not sure why. |
@jonathanolson and I considered the following strategies:
|
I went with (2) for now since it was extremely simple to implement. @ariel-phet can you please recommend a way to proceed here? We are not sure how much time to spend getting the splash screen to look good on IE. |
@samreid I think i would rather leave it buggy then have no splash/loading screen. A blank screen might make the user think the sim is not loading on a slower internet connection with IE. |
This is out for testing in the aforementioned issue. |
Doesn't really look like there is anything here to test? There was a commit that changed some things, then a second commit that just reverted it. Yes...it is still buggy on IE11 but that's what we expected. @samreid should we mark this deferred (or wontfix) and leave open for later? |
I guess close as wontfix? |
From @BryceAG in phetsims/tasks#764 (comment)
Verified on Win 7 and 8.1, IE11 with this link http://www.colorado.edu/physics/phet/dev/html/energy-skate-park-basics/1.3.1-dev.3/energy-skate-park-basics_en.html
The text was updated successfully, but these errors were encountered: