Add GA4 'filter' events to the analytics docs #4574
Conversation
AshGDS
changed the title
data/analytics/events.yml
Outdated
| - name: filter search | ||
| implemented: true | ||
| priority: low | ||
| description: When a user searches for a link on a page, and the search box is one that filters all the results using CSS instead of HTTP or API requests. |
There was a problem hiding this comment.
Suggestion: When a search box is used to filter a long list of things on a page, by showing relevant matches using JavaScript, rather than fetching a new page of results from the server
There was a problem hiding this comment.
@andysellick I was hesitant to just say JavaScript, since our finder searches use client side JavaScript to update search results. Given this, should I still use your suggestion?
There was a problem hiding this comment.
How about When a search box is used to filter a long list of things on a page, by hiding irrelevant matches, rather than fetching a new page of results from the server ?
There was a problem hiding this comment.
@AshGDS I think you might have updated the wrong bit - I meant this description for filter search. The description for filter link clicks could just be Link clicks in a filter search list.
There was a problem hiding this comment.
@andysellick Sounds good 👍 Have updated both descriptions, should be ready for another review
There was a problem hiding this comment.
Whoops, didn't see your latest comment - one second
|
Thanks @andysellick - should be ready for another review 👍 |
AshGDS
force-pushed
the
ga4-add-filter-docs
branch
2 times, most recently
from
033b28c
to
8dc4982
Compare
AshGDS
force-pushed
the
ga4-add-filter-docs
branch
from
8dc4982
to
7dca7eb
Compare
What
.gitignoreas it tries to addnode_modulesto your commits and.DS_Storeif you've ever worked with images in this repo