Skip to content

Conversation

@microbit-grace
Copy link

No description provided.

@github-actions
Copy link

Preview build will be at
https://review-ml.microbit.org/remove-connect-first-view

@microbit-grace
Copy link
Author

Might need to update walkthrough if the user doesn't connect when they first get to the data samples page?

@microbit-matt-hillsdon
Copy link

image

Whatever we do with tours, we definitely need to do something about this UI state which tells you to do something you can't do.

@microbit-grace
Copy link
Author

microbit-grace commented Sep 26, 2024

Whatever we do with tours, we definitely need to do something about this UI state which tells you to do something you can't do.

Here are options I can think of:

  1. Add a dialog if the user tries to click on the disabled button that tells the user that they need to connect first.
  2. Add another similar-styled step before this step that tells user to connect first. Possibly use the live graph panel space to show arrow and text.

What do you think?

Update Have implemented option 1.

Even though it might be a bit strange for the prompt to hint you to record when you can't, maybe it's ok because the user has had a chance to connect when first starting a new session, but dismissed the dialog.

@microbit-grace microbit-grace changed the title [WIP - DO NOT MERGE] Remove connect first view in data samples page Remove connect first view in data samples page Sep 26, 2024
@microbit-grace microbit-grace changed the title Remove connect first view in data samples page [WIP - DO NOT MERGE] Remove connect first view in data samples page Sep 26, 2024
@microbit-matt-hillsdon
Copy link

We've merged this into #331

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