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

Navigation docs #501

Merged
merged 6 commits into from
Nov 19, 2020
Merged

Navigation docs #501

merged 6 commits into from
Nov 19, 2020

Conversation

paolodamico
Copy link
Contributor

Draft proposal of navigation documentation. Related to PostHog/posthog#1775

@paolodamico
Copy link
Contributor Author

Largest changes for which I'd particularly like to get input:

  1. Merging Actions & Events. From a user call yesterday (using Experiment: Improving actions UX? posthog#1841), seems like a natural step in the mental model. This is now a section of user interactions / user engagement, where events is any interaction and action is a particular interaction or series of interactions.

  2. Should we keep project config in the main nav? Project config can already be accessed from the top bar (see screenshot below), based on the settings available in the project configuration it seems like this page will seldom be accessed, and with the command palette users may discover the exact settings they need anyways.

image

image

  1. Should we keep retention in insights? My bet is that once we get this up to speed with CH and Retention allow selection of A and B event posthog#1344 & Retention cohorts graphs posthog#1936 this will be a hot feature. It also goes to the point of setting standards and helping users understand how to do product analytics, by putting it in a prominent place, we'll nudge more users to measure and optimize this metric.

@Twixes
Copy link
Member

Twixes commented Oct 20, 2020

I feel like settings should be somewhat more prominent. I think that we should keep the sidebar analytics-focused, though making settings just a small icon in the project selectors also seems to obscure them a bit. Not sure yet about alternatives.

@paolodamico
Copy link
Contributor Author

I feel like settings should be somewhat more prominent. I think that we should keep the sidebar analytics-focused, though making settings just a small icon in the project selectors also seems to obscure them a bit. Not sure yet about alternatives.

Yup, how about we A/B test this? A -> Having the small icon + command palette only. B -> Small icon, command palette & main icon in sidebar.

@paolodamico paolodamico mentioned this pull request Oct 26, 2020
Copy link
Contributor

@yakkomajuri yakkomajuri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This sounds to me like it's more Handbook than Docs, since it discusses the rationale for adding new items to the nav and all

@paolodamico
Copy link
Contributor Author

Re @yakkomajuri well I would argue this is documenting something related to the app, the handbook feels more for company stuff, but I'm not too set on having it in any particular place.

@yakkomajuri
Copy link
Contributor

yakkomajuri commented Nov 10, 2020

I see the value of this in the Docs, but bits like this:

Adding navigation items

When adding features, the following guidelines can be followed to understand where to place the navigation items.
...

Are instructions for us on how to do things, which generally goes on the handbook

@yakkomajuri yakkomajuri merged commit d65ce99 into master Nov 19, 2020
@yakkomajuri yakkomajuri deleted the feat-201004 branch November 19, 2020 10:49
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.

3 participants