fix(tracker): send unload event on page hidden instead #139
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This removes all the extra logic used to calculate how long the user spent with the page hidden from view, and then calculating the full length spent subtracting time hidden. In theory, this seemed like a great idea, but in practice there were too many edge cases where the browser would send a ridiculously large time spent on page value that isn't representative of the user.
Now the tracker sends the
unload
event on the first time the page is hidden and considers that time spent on page. This should reduce the number of anomalies in data and not skew lesser visited pages as much.