Skip to content
Snippets Groups Projects
Verified Commit 8f031831 authored by Nik | Klampfradler's avatar Nik | Klampfradler
Browse files

Allow maskable icons for non-masked use; mitigates #640

parent b5cea4a8
No related branches found
No related tags found
1 merge request!951Resolve "All icons referred in the manifest have purpose: "maskable" set"
Pipeline #53096 passed
......@@ -15,8 +15,14 @@ Fixed
* Migration added in 2.7.2 did not work in all scenarios
* [Dev] Field change tracking API for Person was broken in 2.7.2
* [OAuth] Automatic clean-up of expired OAuth tokens could fail
* Allow maskable icons for non-masked use
* Add missing documentation
Known issues
~~~~~~~~~~~~
* Maskable and non-masked icons *purpose) any cannot be separated
`2.7.2`_ - 2022-01-31
---------------------
......
......@@ -175,7 +175,7 @@ class ManifestView(View):
{
"src": favicon_img.faviconImage.url,
"sizes": f"{favicon_img.size}x{favicon_img.size}",
"purpose": "maskable" if prefs["theme__pwa_icon_maskable"] else "any",
"purpose": "maskable any" if prefs["theme__pwa_icon_maskable"] else "any",
}
for favicon_img in pwa_imgs
]
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment