Skip to content

Added Onboarding page#17

Closed
Skbonde05 wants to merge 1 commit intobrowseping:mainfrom
Skbonde05:main
Closed

Added Onboarding page#17
Skbonde05 wants to merge 1 commit intobrowseping:mainfrom
Skbonde05:main

Conversation

@Skbonde05
Copy link

Problem

New users had no guidance after installing the extension, leading to confusion and early drop-off.

Solution

Added a lightweight onboarding flow shown once after install that explains:

  • What BrowsePing does
  • How presence tracking works
  • Privacy controls

Implementation

  • Conditional onboarding rendering in App.tsx
  • New Onboarding component in popup
  • Persistent state using chrome.storage
  • No backend or permission changes

Impact

Improves first-time user experience and activation without affecting existing users.

@akash-kumar-dev
Copy link
Member

The onboarding page is not styled and doesn't follow the current UI structure. Additionally, there's a state issue where it only displays once, even after logging out and reopening the popup.
image


const steps = [
{
title: '👀 What is BrowsePing?',
Copy link
Member

Choose a reason for hiding this comment

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

Avoid using inline icons. Please use the React Icons library and follow the existing code structure for incorporating them.

@Skbonde05
Copy link
Author

I will resolve this.

{showOnboarding ? (
<Onboarding
onFinish={() => {
chrome.storage.local.set({ hasSeenOnboarding: true });
Copy link
Member

Choose a reason for hiding this comment

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

This current implementation only allows the onboarding to be seen once. We need a solution that displays the onboarding when the user is in a logged-out state. This requires revisiting the onboarding content to ensure it's clear and engaging.

@akash-kumar-dev
Copy link
Member

I will resolve this.

Please work on a new branch and feel free to open a fresh PR with good commit standards. Please review the contributing.md file.

@Skbonde05
Copy link
Author

Yes

};

return (
<div style={{ padding: 16 }}>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do use tailwind css and match the current UI structure

@akash-kumar-dev akash-kumar-dev added the invalid This doesn't seem right label Jan 3, 2026
@akash-kumar-dev
Copy link
Member

#16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

invalid This doesn't seem right

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments