Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Scroll Depth Release #5120

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Scroll Depth Release #5120

wants to merge 4 commits into from

Conversation

RobertJoonas
Copy link
Contributor

Changes

This PR can be merged once the scroll depth script has been collecting data for a couple of days already. Will be keeping it as a draft until then.

Companion PRs:

  1. docs: scroll depth docs#573
  2. website blog post: Create Blog “2025-01-17-the-easiest-scroll-depth-tracking-ever-–-exclusively-in-plausible” website#735

This PR:

  • updates changelog about the new features (also about the bfcache fix that is currently under the pageleave extension)
  • Adds info paragraphs with documentation links into the "Add goal" form for both pageview and scroll goals
  • Makes the scroll_depth metric available in the public API

Tests

  • Automated tests have been added

Changelog

  • Entry has been added to changelog

Documentation

  • PRs open as mentioned above

Dark mode

  • The UI has been tested both in dark and light mode

@RobertJoonas RobertJoonas marked this pull request as draft February 26, 2025 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@RobertJoonas