-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Experiment: Improving actions UX? #1841
Conversation
Loving the UX improvements!
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Action creation in new version crashes for me when adding a new match group of type Autocapture or Page view (Custom event works fine).
Besides that issue, the new UX seems clearer and more polished to me. Though @mariusandra raised very fair points.
Agree with Marius' points. In addition: WordingWe're now referring to actions as "actions" and "tags". Is this confusing? We discussed "Tags" and we felt that they don't cover the fact you can add logic to them. However, "Power Tags" seems cheesy. Having thought harder, I feel like "Filters" (with everything nested under "Events" in the left menu structure) would be better. There is some conflict with filters on the graphs ToolbarI think the toolbar's UX could be better, given many users could go there to create an action. "Inspect" is actually the best way to create a new action, whereas "Actions" just displays the existing ones. I feel like these should both be in the same place as "Actions". "Inspect" makes me feel like it's going to let me see the xpath but nothing else. If we do this, we should change the wording here (Whilst here, perhaps we should just remove stats from the toolbar for now as that adds confusion whilst it doesn't do anything) Insights pageI'd suggest this shows 'Raw Events' as selected by default (since that will work for everyone), then 'Filtered Events' as an alternate option if we make the wording change above. |
Thanks @Twixes, @mariusandra & @jamesefhawkins for the thorough feedback, great catches!
|
Changes
This PR introduces an experiment (A/B test) to attempt to address #1755. Almost every change in this PR (except for a few tangential adjustments are behind a feature flag). The following specific changes are introduced:
Actions page
Pageviews
action that may be a bit confusing.Insights
Other
Checklist
We may need to sort out #1840 as this PR client-side PostHog analytics (without pageviews or autocapture) to support feature flags on development.