Skip to content
This repository has been archived by the owner. It is now read-only.

Refine Time Spent Calculation #7486

Closed
jonathansampson opened this issue Mar 4, 2017 · 3 comments
Closed

Refine Time Spent Calculation #7486

jonathansampson opened this issue Mar 4, 2017 · 3 comments

Comments

@jonathansampson
Copy link
Collaborator

@jonathansampson jonathansampson commented Mar 4, 2017

  • Did you search for similar issues before submitting this one?
    Yes

  • Describe the issue you encountered:
    Time Spent is presently too lenient. One site, left open when I walked away from my machine for the evening, was granted a Time Spent amount of over 10 hours. Other users report similarly skewed reports for domains in their list. See https://twitter.com/givanse/status/837495337455501313.

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Desktop

  • Brave Version (revision SHA):
    0.13.x

  • Steps to reproduce: (Unconfirmed)

    1. Open multiple tabs for a single domain
    2. Leave browser open, and leave computer
  • Actual result:
    Brave would factor in some length of inactivity when calculated Time Spent on a particular page.

  • Expected result:
    Brave appears to focus more on how long the page was open, rather than how long the user was interacting with the page.

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    N/A

  • Is this an issue in the currently released version?
    Yes

  • Can this issue be consistently reproduced?
    Yes

@cndouglas
Copy link

@cndouglas cndouglas commented Mar 4, 2017

0.13.5 has already shipped. 😄 Moving to 0.13.6.

@cndouglas cndouglas modified the milestones: 0.13.6, 0.13.5 Mar 4, 2017
@jonathansampson
Copy link
Collaborator Author

@jonathansampson jonathansampson commented Mar 4, 2017

@Liunkae Haha. My mistake. Thank you for correcting.

@mrose17
Copy link
Member

@mrose17 mrose17 commented Mar 4, 2017

appears to be a duplicate of #6826

@mrose17 mrose17 closed this Mar 4, 2017
@luixxiul luixxiul added the duplicate label Mar 4, 2017
@luixxiul luixxiul removed this from the 0.13.6 milestone Mar 4, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.