docs(web-player): add PWA and add-to-home-screen guide#554
Conversation
Document built-in PWA support and platform-specific steps for adding the player to the home screen.
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://thankful-water-0a297bf00-554.eastasia.1.azurestaticapps.net |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d05f8d9883
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| ## PWA 支持与添加到主屏幕 | ||
|
|
||
| 内置 Web 播放器支持 PWA（Progressive Web App）。你可以将播放器页面添加到设备主屏幕，像打开原生应用一样一键进入，获得全屏、沉浸式的观看体验。 |
There was a problem hiding this comment.
This says the player supports PWA, but the shipped player has no linked web app manifest: rg 'manifest|webmanifest' web-ui src/embedded_web_data.h finds no manifest entry, and web-ui/player.html only sets mobile/Apple meta tags. MDN's installability guide says a web app must include a manifest to be a PWA/installable (https://developer.mozilla.org/en-US/docs/Web/Progressive_web_apps/Guides/Making_PWAs_installable#the_web_app_manifest), so Android/desktop users won't get the documented promoted install/banner flow or the advertised app name/icon metadata; please add and embed the manifest or describe this as a basic Add to Home Screen shortcut instead.
Useful? React with 👍 / 👎.
Clarify iOS and desktop add-to-home-screen paths, document LG webOS support, and sync the English translation.
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://thankful-water-0a297bf00-554.eastasia.1.azurestaticapps.net |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7355f8d0db
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| 3. 选择 **添加到主屏幕** 或 **安装应用** | ||
| 4. 按提示确认安装 | ||
|
|
||
| 部分浏览器还会在地址栏或页面底部显示安装横幅，点击 **安装** 或 **添加** 即可完成。 |
There was a problem hiding this comment.
Separate from the missing-manifest comment, even if a manifest is added, this install-banner flow still won't appear for the documented LAN URLs such as http://192.168.1.1:5140/player: Chromium installability requires HTTPS or localhost (https://web.dev/articles/install-criteria), and this daemon's --listen endpoint is plain HTTP unless users add a TLS reverse proxy. Please call out the HTTPS/reverse-proxy requirement or describe Android/desktop HTTP use as a browser shortcut instead, otherwise users following these steps will not see Install app/the banner.
Useful? React with 👍 / 👎.
1 participant
Summary
Test plan
/guide/web-player/en/guide/web-player