launchpad-reviewers team mailing list archive

[Colin Watson <mp+423966@xxxxxxxxxxxxxxxxxx>]

Diff comments:

> diff --git a/lib/canonical/launchpad/images/site.webmanifest b/lib/canonical/launchpad/images/site.webmanifest
> new file mode 100644
> index 0000000..6c28a17
> --- /dev/null
> +++ b/lib/canonical/launchpad/images/site.webmanifest
> @@ -0,0 +1,19 @@
> +{
> +    "name": "",

It's not an image, indeed, but I think it probably makes sense to put it in here anyway:

 * It's better to avoid putting things in the top level, because anything we put there clashes with the namespace for projects/distributions and may require manual blocklisting there (https://launchpad.net/+nameblacklist), which is cumbersome.
 * It's certainly closely associated with images even if it isn't technically one itself.
 * In `lib/lp/app/browser/configure.zcml`, `/@@/` is referred to as the "resources namespace".

> +    "short_name": "",
> +    "icons": [
> +        {
> +            "src": "/@@/android-chrome-192x192.png?v=2022",
> +            "sizes": "192x192",
> +            "type": "image/png"
> +        },
> +        {
> +            "src": "/@@/android-chrome-512x512.png?v=2022",
> +            "sizes": "512x512",
> +            "type": "image/png"
> +        }
> +    ],
> +    "theme_color": "#ffffff",
> +    "background_color": "#ffffff",
> +    "display": "standalone"
> +}


-- 
https://code.launchpad.net/~lgp171188/launchpad/+git/launchpad/+merge/423966
Your team Launchpad code reviewers is requested to review the proposed merge of ~lgp171188/launchpad:update-favicon-sizes into launchpad:master.