LotusGuild/cinny
3
0
Fork 0
Code Issues 42 Pull Requests 2 Actions Packages Projects Releases Wiki Activity

No warning when closing the tab or reloading during an active call #58

New Issue
Closed
opened 2026-09-12 01:51:18 -04:00 by jared · 0 comments
jared commented 2026-09-12 01:51:18 -04:00
Owner
Copy Link
Copy Source

Severity: low · Type: ux · Confidence: high

Location: src/app/components/CallEmbedProvider.tsx (no beforeunload handler anywhere — grep beforeunload src only matches usePresenceUpdater.ts:153's pagehide)

Problem

nothing registers a beforeunload guard while a call is joined. Ctrl+W / Ctrl+R / accidentally closing the window drops the user out of the call with no confirmation, and the clean-leave then depends entirely on EC's own unload handler racing the teardown.

How to trigger

join a call, press Ctrl+R.

Suggested fix

register a beforeunload listener while callEmbed && joined that sets event.preventDefault(), and remove it on hangup/dispose.

Filed from the September 2026 client audit (branch lotus @ 4bea4895).

**Severity:** low · **Type:** ux · **Confidence:** high **Location:** `src/app/components/CallEmbedProvider.tsx` (no `beforeunload` handler anywhere — `grep beforeunload src` only matches `usePresenceUpdater.ts:153`'s `pagehide`) ### Problem nothing registers a `beforeunload` guard while a call is joined. Ctrl+W / Ctrl+R / accidentally closing the window drops the user out of the call with no confirmation, and the clean-leave then depends entirely on EC's own unload handler racing the teardown. ### How to trigger join a call, press Ctrl+R. ### Suggested fix register a `beforeunload` listener while `callEmbed && joined` that sets `event.preventDefault()`, and remove it on hangup/dispose. --- _Filed from the September 2026 client audit (branch `lotus` @ 4bea4895)._
jared added this to the Audit 2026-09 · Medium & Low milestone 2026-09-12 01:51:18 -04:00
jared added the priority: lowuxarea: calls labels 2026-09-12 01:51:18 -04:00
jared self-assigned this 2026-09-12 01:51:18 -04:00
jared closed this issue 2026-09-12 20:29:01 -04:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
a11y
Accessibility: keyboard, screen reader, contrast, motion
 area: appearance
Client area: appearance
 area: auth-session
Client area: auth-session
 area: build-ci
Client area: build-ci
 area: calls
Client area: calls
 area: desktop
Client area: desktop
 area: media
Client area: media
 area: messaging
Client area: messaging
 area: mobile
Client area: mobile
 area: moderation
Client area: moderation
 area: navigation
Client area: navigation
 area: notifications
Client area: notifications
 area: settings
Client area: settings
 area: threads
Client area: threads
 bug
Something is not working
 dependencies
Third-party package versions and advisories
 docs
README / LOTUS_* docs wrong or missing
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 needs-human-review
performance
Re-render storms, leaks, heavy work on hot paths
 planning
priority: critical
Data loss, security hole, or crash on a main path
 priority: high
Broken feature or serious usability problem
 priority: low
Minor issue or polish
 priority: medium
Wrong behaviour in an edge case or notable degradation
 qa
Manual QA: shipped, needs a human in a real environment
 question
More information is needed
 research
security
XSS, unsafe URLs, data leaks, auth/session
 tech-debt
Code health, dead code, fragile patterns
 ux
Usability or visual inconsistency
 wontfix
This won't be fixed
No labels area: calls priority: low ux
Milestone
No items
No Milestone Audit 2026-09 · Medium & Low
Projects
Clear projects
No projects
Assignees
Clear assignees
cole (Cole Keller) jared (Jared Vititoe) nathan (Nathan Vititoe)
No Assignees jared
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: LotusGuild/cinny#58