-
Notifications
You must be signed in to change notification settings - Fork 217
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.
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
Analytics event: BACK_TO_TOP
#2276
Comments
Added the "🧹 status: ticket work required" label to convert the issue into the same format as the other Analytics event PRs. |
BACK_TO_TOP
hi @fcoveram, i'd be interested in working on this! curious, what should we track in this event? i was thinking:
|
Great, more than welcome. I will refer to @sarayourfriend as I think added the technical info in the main comment. |
Search type does not need to be stored in the payload, it can be inferred from the path (
|
The description is updated with the payload parameters to add and I've assigned the issue to you @nevvada. Let us know if you need any help 👍 |
Description
The UI component assumes that users need to go back to the top part of Openverse interface, and this change would help us to understand whether that is true or if any other action derivates from this search flow.
Steps:
BACK_TO_TOP
to theEvents
type:openverse/frontend/src/types/analytics.ts
Line 12 in 839418f
query
andpage
payload for the query term and requested page depth respectively. Additionally,scrolledPixels
derived fromdocument.getElementById('main-page').scrollTop
(or, preferably a Vue ref on that element instead) and similarlymaxScroll
derived from.scrollTopMax
off the same#main-page
element.useAnalytics
composable when the back to top button is clicked. It should be added to https://github.com//WordPress/openverse/blob/HEAD/frontend/src/components/VScrollButton.vue#L47-L50Additional context
Screenshot of the UI component mentioned
Additional information
VHomeGallery.vue
The text was updated successfully, but these errors were encountered: