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

Guided tour for new users #4690

Merged
merged 14 commits into from
Jul 3, 2024
Merged

Guided tour for new users #4690

merged 14 commits into from
Jul 3, 2024

Conversation

pfrazee
Copy link
Collaborator

@pfrazee pfrazee commented Jun 28, 2024

Adds a guided tour for new users

  • Works on all platforms
  • Handles variable font sizes
  • Trigger when a new account is created
  • Put behind a gate to measure impact
  • Make sure VoiceOver is working correctly

iOS/Android

CleanShot.2024-06-27.at.16.38.22.mp4

Web

CleanShot.2024-06-27.at.16.36.06.mp4

Copy link

render bot commented Jun 28, 2024

Copy link

github-actions bot commented Jun 28, 2024

Old size New size Diff
6.56 MB 6.58 MB 20.6 KB (0.31%)

@pfrazee pfrazee changed the title Guided tour for new users [WIP] Guided tour for new users Jun 28, 2024
@mackuba
Copy link

mackuba commented Jun 29, 2024

👀

@pfrazee
Copy link
Collaborator Author

pfrazee commented Jul 2, 2024

@estrattonbailey with the sick assist on a11y, very much appreciated


React.useEffect(() => {
if (!isWeb) return
document.body.style.overflow = 'hidden'
Copy link
Collaborator

Choose a reason for hiding this comment

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

We have useWebBodyScrollLock for this, I think it should be more reliable because it also deals with ref counting (if there's overlap) and handles scrollbar gutter

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done

Copy link
Collaborator

@gaearon gaearon left a comment

Choose a reason for hiding this comment

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

lgtm with a nit

@pfrazee pfrazee merged commit a3d4fb6 into main Jul 3, 2024
6 checks passed
@pfrazee pfrazee deleted the paul/guided-tour branch July 3, 2024 04:25
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.

4 participants