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

Homepage guided tour #173

Merged
merged 12 commits into from
Sep 12, 2023
Merged

Homepage guided tour #173

merged 12 commits into from
Sep 12, 2023

Conversation

FaithDaka
Copy link
Collaborator

@FaithDaka FaithDaka commented Jul 27, 2023

Description

  • Implemented service to facilitate communication between the app-open-prompt and the homepage. Tour currently appears after prompt is dismissed.
  • Implemented guided tour using intro.js library. Each tile is described.

Discussion

  • Implementing the tour after the user clicks to "Continue in app"

Preview

Link to app preview if relevant

Screenshots / Videos

Guided.Tour.mp4

@FaithDaka FaithDaka self-assigned this Jul 27, 2023
@FaithDaka FaithDaka marked this pull request as ready for review July 31, 2023 09:14
Copy link
Collaborator

@chrismclarke chrismclarke left a comment

Choose a reason for hiding this comment

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

Many thanks @FaithDaka and apologies for the slow review.

The implementation looks great. I've added two more commits to refactor the code to a separate shared tour service, and added a little bit of styling to the tooltip and start tour buttons (screenshots below)

We've had a couple calls with UoR this week and they're particularly keen to make a lot of use of the tour functionality for direct-to-farmer interactive tutorials, so really nice to have this in as a base to work from

localhost_4200_(PICSA-Tablet)

localhost_4200_(PICSA-Tablet) (1)

@chrismclarke chrismclarke merged commit 8c74e0e into main Sep 12, 2023
2 checks passed
@chrismclarke chrismclarke deleted the ft-homepage-intro branch September 12, 2023 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants