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

Toolbar experience for non-toolbar users #2777

Closed
paolodamico opened this issue Dec 15, 2020 · 21 comments
Closed

Toolbar experience for non-toolbar users #2777

paolodamico opened this issue Dec 15, 2020 · 21 comments
Labels
enhancement New feature or request feature/toolbar Feature Tag: Toolbar P2 Semi-urgent, non-breaking, affects UX but functional stale

Comments

@paolodamico
Copy link
Contributor

Is your feature request related to a problem?

There are some cases where users are unable to use the toolbar. The most common one is if you don't have the snippet/JS library (or any front-end events at all). If this is the case, it's a bit distracting/annoying and even concerning to see the floating toolbar prompt come up.

Describe the solution you'd like

If there are no frontend events which the toolbar can use, do not show the floating thing. We can still keep the menu item as a CTA to take actions for the toolbar to work.

Related to the above, we should show a CTA with clear instructions on how to address enabling the toolbar (i.e. likely adding the snippet or JS library to the frontend, or enabling $autocapture if disabled).

Additional context

Related to customer call

Thank you for your feature request – we love each and every one!

@paolodamico paolodamico added enhancement New feature or request feature/toolbar Feature Tag: Toolbar labels Dec 15, 2020
@macobo
Copy link
Contributor

macobo commented Dec 17, 2020

Idea: on clicking x for the toolbar, prompt to dismiss it 'forever' (for this user), then say "you can always re-enable it from settings". :) And add a toggle under settings.

@posthog-bot
Copy link
Contributor

This issue hasn't seen activity in 6 months. Is it still relevant?

@paolodamico paolodamico added core-experience P2 Semi-urgent, non-breaking, affects UX but functional and removed stale labels Jul 21, 2021
@clarkus
Copy link
Contributor

clarkus commented Jul 27, 2021

Related #4268 - There is a toggle for disabling the toolbar here. Would that qualify as a solution here or is the core problem exposing this setting in the context of the toolbar?

@paolodamico
Copy link
Contributor Author

User reported wanting to disable the toolbar for themselves but not for the entire project which wouldn't be covered by this.

@clarkus
Copy link
Contributor

clarkus commented Jul 29, 2021

User reported wanting to disable the toolbar for themselves but not for the entire project which wouldn't be covered by this.

So given that, there should be two settings. One to enable the toolbar for the project, but secondarily to enable it for a specific user account?

@paolodamico
Copy link
Contributor Author

So I’m actually not sure that’s the best thing to do, just wanted to put all the context I had forward. Perhaps one option is to have a single project setting but we let you permanently dismiss the toolbar when you click the close button. If you permanently dismissed the toolbar, we give you an option to re-enable it in settings.

@macobo
Copy link
Contributor

macobo commented Jul 30, 2021

Idea: "x" on the hedgehog on site. If clicked, asks if to permanently disable, but says you can always re-enable by launching it via app.posthog.com.

@paolodamico
Copy link
Contributor Author

Yup 100%, exactly what I meant. Though if you permanently dismiss it I think it should only apply for your user not your project right ?

@clarkus
Copy link
Contributor

clarkus commented Jul 30, 2021

Yeah my comment was more about the places where a user can change this in settings. It'd be totally fine to changes those settings or to prompt to change them when interacting with the toolbar. I was mostly thinking about the other case where the toolbar is enabled for the project, but disabled for a user, and a user needed to know how to re-enable the toolbar. A single setting at the project level wouldn't be sufficient for that.

@paolodamico
Copy link
Contributor Author

Well that ends up being connected to what we do with the settings page but indeed we need a separate user-level setting for that. One UX thing that I think makes sense is only showing that extra config if you have permanently disabled the toolbar (i.e. you can only disable by clicking the close button)

@clarkus
Copy link
Contributor

clarkus commented Aug 3, 2021

Here's an illustration of a possible solution. In this case, on dismiss we prompt the user to disable the toolbar for their User Account. Thoughts?

Disable toolbar prompt

@paolodamico
Copy link
Contributor Author

Yes, pretty close though I think we need to have 3 buttons: "Cancel", "Dismiss forever" and "Close for this session" (primary) [exact copy to be adjusted]. It might also be worth to have a "Don't ask me again" checkbox for option 3 (primary).

@clarkus
Copy link
Contributor

clarkus commented Aug 5, 2021

Is the task dismissing the toolbar or disabling it? To me, dismiss means stop showing this, but disable means never show it again (unless I turn it back on in settings). The close action on the toolbar is the "dismiss" portion, and the follow up prompt is the "for how long" portion of the interaction.

👍 on the "don't ask me again" checkbox.

Screen Shot 2021-08-05 at 8 39 21 AM

@paolodamico
Copy link
Contributor Author

Yes agreed! Disabling is "permanent", dismissing is temporary (i.e. this session), but you trigger either by clicking on the same close icon button, that's why I was suggesting the 3 buttons. The thing is that once you have the dialog, clicking on cancel to only dismiss temporarily seems confusing, as cancel could simply entail go back and don't actually close the toolbar.

@clarkus
Copy link
Contributor

clarkus commented Aug 5, 2021

Good catch. We're not letting them cancel out of the dismissal, we're just confirming it. What do you think of these changes?

Screen Shot 2021-08-05 at 1 57 32 PM

@paolodamico
Copy link
Contributor Author

Love it! One final suggestion if I may. Feels like the primary action should be just dismiss on this session rather than fully disable (is at least something we'd like to encourage I think). Wdyt?

@clarkus
Copy link
Contributor

clarkus commented Aug 5, 2021

Sure! Screen Shot 2021-08-05 at 2 06 24 PM

Available in figma at https://www.figma.com/file/9yWtngNb1AIuf6KmXaEPJA/App-doodles?node-id=900%3A140805

@clarkus
Copy link
Contributor

clarkus commented Aug 12, 2021

Related #2370

@corywatilo
Copy link
Contributor

My thoughts on disabling vs dismissing:

I don't know if we need to disable it, to the extent you have to go back into Settings to re-enable. What seems like a simpler UX is, once you click X to dismiss it, it just doesn't re-appear until you go to the Toolbar and re-launch it on the domain where it was previously dismissed. Then it would stick around until you X it out again.

@posthog-bot
Copy link
Contributor

This issue hasn't seen activity in two years! If you want to keep it open, post a comment or remove the stale label – otherwise this will be closed in two weeks.

@pauldambra
Copy link
Member

once you click X to dismiss it, it just doesn't re-appear until you go to the Toolbar and re-launch it on the domain where it was previously dismissed.

that's how it works now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feature/toolbar Feature Tag: Toolbar P2 Semi-urgent, non-breaking, affects UX but functional stale
Development

No branches or pull requests

6 participants