-
Notifications
You must be signed in to change notification settings - Fork 6k
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
base: dev
Are you sure you want to change the base?
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. |
PWA Support: - Service worker with intelligent caching (stale-while-revalidate for HTML) - Web app manifest for installable PWA experience - iOS meta tags and safe area inset support Mobile UI: - Virtual keyboard detection for proper layout adjustments - Responsive layout improvements - Mobile-friendly server URL detection for remote access Web Push Notifications: - Push event handler in service worker - Notification click handler to focus/open app - Server endpoints for push subscription management - Auto-subscribe when notification permission granted
a18e38b to
7556808
Compare
Resolved conflict in prompt-input.tsx by combining mobile-friendly styling with new provider icons and model variant selector features.
|
Haha thanks for the quick implementation! Looks awesome! |
Summary
Features
Related Issues
Demo
Before
trim.3D3639B6-5A06-4CEC-A581-2E71E71D836D.MOV
After
trim.DC05A0D8-1C92-40CB-8646-1D8C25C2CC2D.MOV