add eventModification debug flag

[scytacki]

No description provided.

[codecov]

Codecov Report

Attention: Patch coverage is 33.33333% with 12 lines in your changes missing coverage. Please review.

Project coverage is 85.99%. Comparing base (3eb9c48) to head (9103bdd).
Report is 129 commits behind head on main.

Files with missing lines	Patch %	Lines
v3/src/lib/debug-event-modification.ts	25.00%	12 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1811      +/-   ##
==========================================
+ Coverage   85.95%   85.99%   +0.04%     
==========================================
  Files         636      637       +1     
  Lines       32739    32757      +18     
  Branches     8572     9203     +631     
==========================================
+ Hits        28140    28170      +30     
+ Misses       4443     4255     -188     
- Partials      156      332     +176     

Flag	Coverage Δ
cypress	73.36% <25.00%> (+0.06%)	⬆️
jest	55.29% <33.33%> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[cypress]

codap-v3    Run #6569

Run Properties:   Passed #6569  •  717841b0da: Merge pull request #1811 from concord-consortium/debug-event-modification

Project	codap-v3
Branch Review	main
Run status	Passed #6569
Run duration	06m 27s
Commit	717841b0da: Merge pull request #1811 from concord-consortium/debug-event-modification
Committer	Scott Cytacki
View all properties for this run ↗︎

---

Test results
Failures	0
Flaky	0
Pending	51
Skipped	0
Passing	245
View all changes introduced in this branch ↗︎

[kswenson]

👍 Looks good -- I suggest adding releasePointerCapture to the events tracked.

[kswenson]

Suggested change

	"setPointerCapture"
	"setPointerCapture",
	"releasePointerCapture"

I think it would be useful to track both to make it clear whether the capture was released or not.

[scytacki]

I'll add it, but I'd also note that it isn't necessary to call releasePointerCapture. According to https://developer.mozilla.org/en-US/docs/Web/API/Element/setPointerCapture the capture will be released automatically when the pointerup event is fired.

[kswenson]

Good to know.