Skip to content
This repository has been archived by the owner on Mar 14, 2024. It is now read-only.

Add manifest fields for PWA install bottom sheet #4476

Merged
merged 1 commit into from
Jan 21, 2021

Conversation

beaufortfrancois
Copy link
Member

@beaufortfrancois beaufortfrancois commented Jan 19, 2021

Following GoogleChromeLabs/squoosh#933 and #4502, this PR adds standardized manifest fields for PWA install bottom sheet such as description, lang, categories, and screenshots so that the PWA install bottom sheet looks great. See screenshot below.

Note that it currently requires chrome://flags/#mobile-pwa-install-use-bottom-sheet flag in Chrome Canary on Android.

@beaufortfrancois beaufortfrancois added the DO NOT MERGE Actively working on but experimental label Jan 19, 2021
@beaufortfrancois beaufortfrancois requested a review from a team as a code owner January 19, 2021 09:39
@google-cla google-cla bot added the cla: yes Contributor has signed the CLA label Jan 19, 2021
@github-actions
Copy link

github-actions bot commented Jan 19, 2021

Alex Recommends Report

Alex recommends the following language changes, but Alex is a regular expression based algorithm, so take them with a grain of salt.

✨ 🚀 ✨ Nothing to Report ✨ 🚀 ✨

@netlify
Copy link

netlify bot commented Jan 19, 2021

Deploy preview for web-dev-staging ready!

Built with commit b13928b

https://deploy-preview-4476--web-dev-staging.netlify.app

@beaufortfrancois beaufortfrancois marked this pull request as draft January 19, 2021 09:47
@beaufortfrancois beaufortfrancois marked this pull request as ready for review January 21, 2021 10:54
@beaufortfrancois beaufortfrancois changed the title [WIP] Add manifest fields for PWA install bottom sheet Add manifest fields for PWA install bottom sheet Jan 21, 2021
@beaufortfrancois beaufortfrancois removed the DO NOT MERGE Actively working on but experimental label Jan 21, 2021
@beaufortfrancois
Copy link
Member Author

@robdodson Can you review?

@robdodson robdodson merged commit 676a8dd into GoogleChrome:master Jan 21, 2021
@beaufortfrancois beaufortfrancois added the content update for issues that do not require new content (only for updates to existing content) label Feb 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes Contributor has signed the CLA content update for issues that do not require new content (only for updates to existing content)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants