-
Notifications
You must be signed in to change notification settings - Fork 225
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
Broken on Safari 9.1.1 on Mac #45
Comments
hi @t1
and then, perhaps use:
this should be done for all window properties that are retrieved for calculations. |
@orizens: I'm quite new to js, and try to stick to things like Angular ;-) But you seem to know exactly what would have to be done. Do you think you will be able to find the time to do it? And maybe even can say, when this might be? |
Hi, what's the status of this bug? This looks like a quickfix according to your answer. It's nicer to have this fixed in the master branch because I believe most of us are using npm to install the dependencies and it gives us the compiled version, which is not very nice to fix. |
hi @freezing |
Thanks a lot for fixing this. It's working well :) |
While it works fine on Chrome, it just doesn't load more on Safari. Not in the plunkr, nor in the echoes.
Looking at the other issues, you seem to be testing with Safari, too confused
The text was updated successfully, but these errors were encountered: