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.

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

feat(web-analytics): Add info icons on all web analytics sections #26513

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

Conversation

rafaeelaudibert
Copy link
Contributor

Problem

A user complained that they usually have a hard time understanding what each of the sections of the web analytics product mean. This will always happen for less-technical users and/or users who are learning how to track web analytics for the first time in their lives.

Changes

Most of the sections have an info button explaining what that section is about AND referring them to our docs website - except for the extremely obvious ones, such as Devices and countries.

I'm extremely open to copy updates.

Here are some screenshots, but you can check all of the changes by looking at the code as the most important thing to review is the copy for all of them.

image image image

Does this work well for both Cloud and self-hosted?

Yes

How did you test this code?

Manual testing (UI)

@rafaeelaudibert rafaeelaudibert changed the title Add info icons on all web analytics sections feat(web-analytics): Add info icons on all web analytics sections Nov 28, 2024
Copy link
Contributor

Size Change: +105 B (+0.01%)

Total Size: 1.16 MB

ℹ️ View Unchanged
Filename Size Change
frontend/dist/toolbar.js 1.16 MB +105 B (+0.01%)

compressed-size-action

@robbie-c
Copy link
Member

Looks good! My one piece of feedback, feel free to ignore, is that we could improve the discoverability of custom channel types by pulling the link out of the tooltip, and have it always visible with a cog icon, similar to editing the path cleaning filters.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants