-
Notifications
You must be signed in to change notification settings - Fork 7.1k
feat: add PWA support and mobile UI improvements #7258
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
Conversation
|
The following comment was made by an LLM, it may be inaccurate: No duplicate PRs found |
8e14998 to
1f1089e
Compare
70a44b5 to
e016d06
Compare
e016d06 to
67cc211
Compare
d72695d to
0da9180
Compare
e359812 to
085ad3d
Compare
|
Great work on this PWA support PR, this is really valuable! ✨ |
22c516f to
a18e38b
Compare
|
Thanks for the suggestion! I've added Web Push notification support in the latest commit:
This allows notifications to be delivered even when the PWA is closed. |
a18e38b to
7556808
Compare
|
Haha thanks for the quick implementation! Looks awesome! |
- Add remote access support for mobile via Tailscale (serverUrl, port detection) - Add SyncGate loading component for directory layout - Wire up question tool respond/reject handlers in DataProvider - Pass sessionID/callID to tool props for question interactivity - Add responsive padding in prompt-input for mobile - Add scroll-to-bottom button and auto-scroll improvements - Add regression tests for mobile features (178 tests)
7338a04 to
08103a5
Compare
|
Closing in favor of a more targeted PR with just the mobile UX improvements (scroll-to-bottom, archive button visibility, responsive spacing). The question tool support was already merged in #8232. |
Summary
Features
Related Issues
Demo
Before
trim.3D3639B6-5A06-4CEC-A581-2E71E71D836D.MOV
After
trim.DC05A0D8-1C92-40CB-8646-1D8C25C2CC2D.MOV