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

Improve UX of "CLI Not Installed" #187

Open
jbyrum-stripe opened this issue Mar 11, 2021 · 7 comments
Open

Improve UX of "CLI Not Installed" #187

jbyrum-stripe opened this issue Mar 11, 2021 · 7 comments

Comments

@jbyrum-stripe
Copy link

When the CLI is not installed:

  • Events pane items should be in disabled state -- user should not, for example, be able to click on the "Start webhooks listening" item in the "Events" pane and launch the functionality (only to later find out it doesn't work)
  • Logs pane item should be in a disabled state
  • Relevant items in the command palette should be discoverable (i.e., don't remove them), but disabled
  • Explore a more prominent placement for the "Install CLI dialog" than the bottom right hand corner. Is it possible to make the dialog more prominent?
  • "Read instructions on how to install Stripe CLI" should deep link to: https://stripe.com/docs/stripe-cli#install
  • Ensure tracking of when a user tries to interact with a disabled item (not sure what flexibility we have here). We should, for example, to be able pull the # of times a disabled item was clicked / hovered over.

Bonus points:

  • On hover of a disabled item, show an onhover menu with a description of the functionality. For example, if the user hovered over "Start webhooks listening", show the message: "Install the CLI to enable the forwarding of webhook events to your local machine". The purpose of this functionality is to entice users to actually install the CLI.
@vcheung-stripe
Copy link
Collaborator

vcheung-stripe commented Mar 12, 2021

@auchenberg
Copy link

@vcheung-stripe @jbyrum-stripe

The new "welcome alert" is quite disruptive to the workflow of opening VS Code without the Stripe CLI installed. Here's a screenshot of me opening VS Code, and the first thing I see is the Stripe alert. Notice that I'm the file explorer, and I haven't even activated the Stripe view in the sidebar. This probably happens because we have changed the extension to activate on * for some reason.

I think we should re-consider this UX, and instead explore #57 which would provide a much less in-your-face-UX via WelcomeViews. Example docs: https://code.visualstudio.com/api/references/contribution-points#contributes.viewsWelcome

vscode_activation_prompt

@richardm-stripe
Copy link
Contributor

richardm-stripe commented Apr 28, 2021

We've had a user reach out about the aggressiveness of the reminder here: stripe/stripe-php#1118, another point in favor of revisiting this UX.

@gracegoo-stripe
Copy link
Contributor

gracegoo-stripe commented Apr 28, 2021

Notice that I'm the file explorer, and I haven't even activated the Stripe view in the sidebar. This probably happens because we have changed the extension to activate on * for some reason.

Thanks for reporting this!

We're not activating on * but we did have some non-stripe specific events on there. I like the idea of pre-loading the extension but the prompts that end up coming up for the user is super annoying, especially if they aren't even doing anything Stripe related at the moment, and will likely just encourage them to uninstall. It also doesn't look like the loading is heavy enough to notice if we don't pre-load it.

#219

gracegoo-stripe added a commit that referenced this issue Apr 28, 2021
@ossigeno
Copy link

ossigeno commented May 5, 2021

I have uninstalled this extension. I already have Stripe CLI installed by way of Homebrew. The constant popups are madness.

@jbyrum-stripe
Copy link
Author

@gracegoo-stripe @vcheung-stripe More feedback from our user survey:

"I have the CLI installed but it keeps saying I dont in VS code. Makes the extension unusable. "

"to many alerts i have zero understanding how to use it yet"

@vcheung-stripe
Copy link
Collaborator

I think we should definitely implement welcome views (#57).

In addition, I've noticed on MacOS that the popup has become much more prominent since the Big Sur update. I think that should have already been reason enough to change this UX.

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

No branches or pull requests

6 participants