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

Some browsers prevent page transitions #36

Closed
martindusinberre opened this issue Aug 22, 2019 · 7 comments
Closed

Some browsers prevent page transitions #36

martindusinberre opened this issue Aug 22, 2019 · 7 comments
Assignees
Labels
bug Something isn't working medium medium priority

Comments

@martindusinberre
Copy link
Collaborator

Pim/Danny/Ramun will be able to explain this better than me, but some players had problems transitioning when they were playing in particular browsers (e.g. Firefox).

@christian-bernet
Copy link
Contributor

christian-bernet commented Aug 28, 2019

estimate: 2 hours

@christian-bernet
Copy link
Contributor

@interrogator can you please give us a reproducible example of that?

@interrogator
Copy link
Collaborator

I never saw this happen and don't really know what it is or what browser affected (and am against a great deal of cross-browser testing at this stage). Maybe @witlox saw it on a tester's machine?

@interrogator
Copy link
Collaborator

Pim and I have a feeling that the problems we saw may actually be related to the internet connection problems during the test. Unless anybody can reliably find a Firefox transition problem, the real fix is actually #24 , which should for this reason get priority.

@interrogator interrogator added medium medium priority and removed high high priority labels Sep 3, 2019
@christian-bernet
Copy link
Contributor

I think it is more related to #46.
When the backend is slow and I click multiple times on a page transition button, it results in multiple backend calls, which is bad.

@christian-bernet
Copy link
Contributor

should also be fixed with #46

@martindusinberre
Copy link
Collaborator Author

I'm closing this as @witlox @lfeine @Helena-LiT are checking #46
Please reopen if there still is a problem with #46

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working medium medium priority
Projects
None yet
Development

No branches or pull requests

5 participants