Skip to content

Conversation

daniel-graham-amplitude
Copy link
Collaborator

Summary

Checklist

  • Does your PR title have the correct title format?
  • Does your PR have a breaking change?:

Copy link

promptless bot commented Jul 15, 2025

✅ No documentation updates required.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Click Duration Metric Displays Incorrect Value

The "Last click time" metric is incorrectly calculated. It uses Date.now() - startTime, which displays the elapsed time since the last measurement began, rather than the actual duration of the last completed click. This results in an inaccurate or continuously increasing value. The correct duration, which is already calculated and stored (e.g., in the clickTimes array), should be displayed instead.

test-server/autocapture/performance-test.html#L180-L181

<p>Median time per click: ${medianTime}ms</p>
<p>Last click time: ${startTime > 0 ? (Date.now() - startTime).toFixed(2) : 0}ms</p>

Fix in CursorFix in Web


Comment bugbot run to trigger another review on this PR
Was this report helpful? Give feedback by reacting with 👍 or 👎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant