Skip to content

feat: add page visibility favicon notifications#410

Draft
saurabhhhcodes wants to merge 1 commit into
Kuldeeep18:mainfrom
saurabhhhcodes:codex/page-title-favicon-notifications
Draft

feat: add page visibility favicon notifications#410
saurabhhhcodes wants to merge 1 commit into
Kuldeeep18:mainfrom
saurabhhhcodes:codex/page-title-favicon-notifications

Conversation

@saurabhhhcodes

Copy link
Copy Markdown
Contributor

What changed

  • Added shared page-visibility handling in frontend/main.js so LeadOrbit switches to a "Come back to LeadOrbit!" title when the tab is hidden.
  • Added a favicon badge renderer that draws a small red notification dot on the existing icon.
  • Hooked the CSV import success path to trigger the notification helper when an import completes while the tab is hidden.

Why

  • Users who leave the app in another tab need a lightweight signal that background work finished.
  • A title change plus favicon badge is less intrusive than system notifications and works well across the app shell.

Validation

  • node --check frontend/main.js
  • node --check /tmp/leads-notify-script.mjs
  • git diff --check

Closes #400

@coderabbitai

coderabbitai Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 9ccb2bfe-5749-4faa-ae87-5ac770b628d7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

LO-063 [Easy]: Dynamic Page Titles and Favicon Notifications

1 participant