fix(tracking): passive section view [Fixes #18047]#18050
Open
myelinated-wackerow wants to merge 1 commit intodevfrom
Open
fix(tracking): passive section view [Fixes #18047]#18050myelinated-wackerow wants to merge 1 commit intodevfrom
myelinated-wackerow wants to merge 1 commit intodevfrom
Conversation
myelinated-wackerow
requested review from
minimalsm,
pettinarip and
wackerow
as code owners
✅ Deploy Preview for ethereumorg ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Six TrackedSection wrappers on the new homepage auto-fired section_view events via IntersectionObserver with no user input. Matomo counts events as actions, so the homepage's bounce rate became incomparable with other pages and pre-launch baselines (apparent 56% -> 33% post-launch was largely an instrumentation artifact). Removes the wrappers from app/[locale]/page.tsx, deletes the unused component, and adds a comment on trackCustomEvent flagging it as user-initiated only to prevent recurrence. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
myelinated-wackerow
force-pushed
the
fix/remove-passive-section-view-tracking
branch
from
099f54b to
5e845ad
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Description
The new homepage (PR #17972, launched 2026-04-20) shipped six
TrackedSectionwrappers that auto-firedsection_viewMatomo events via IntersectionObserver, with no user input. These were added for the Feb–Mar A/B test (70e44a5524,a7f5e3e0b2) and were not cleaned up post-launch.Matomo classifies any visit with
nb_actions >= 2as a non-bounce, and events count as actions. Because the homepage auto-fired events on scroll while no other page does, its bounce-rate metric became incomparable to the rest of the site, to its own pre-launch baseline, and to the A/B test's variant A (which showed 26–28% bounce when instrumented identically vs ~56% for the same design without instrumentation). The apparent post-launch improvement from ~56% to ~33% is largely an instrumentation artifact rather than a behavior change.Changes
TrackedSectionwrappers fromapp/[locale]/page.tsx(kpi,savings_carousel,trust_logos,simulator,feature_cards,get_started).src/components/TrackedSection.tsx(no other consumers).trackCustomEventinsrc/lib/utils/matomo.tsflagging it as user-initiated only, with the bounce-rate rationale, to prevent recurrence.User-click tracking is unchanged:
cta_click,modal_select,section_click,cta_swipe,modal_open,modal_closeall stay — every event tied to a real interaction is preserved.Notes
idattributes on each section. Searched.tsx/.ts/.json/.md/.mdxfor#kpi,#savings_carousel,#trust_logos,#simulator,#feature_cards,#get_started— zero matches, so no in-page anchor links break.usehooks-tsstays as a dep — still used byIntersectionObserverReveal.Test plan
section_viewevents fire on scrollcta_click,section_click, etc.) still fire as beforenb_actionsdistribution to revert toward baselineGenerated by Claude (Opus 4.7)
Related issue