Skip to content
Snippets Groups Projects
Commit 72c5cfab authored by Nik | Klampfradler's avatar Nik | Klampfradler
Browse files

Merge branch '640-all-icons-referred-in-the-manifest-have-purpose-maskable-set' into 'master'

Resolve "All icons referred in the manifest have purpose: "maskable" set"

See merge request !951
parents b5cea4a8 8f031831
No related branches found
No related tags found
1 merge request!951Resolve "All icons referred in the manifest have purpose: "maskable" set"
Pipeline #53164 passed
Pipeline: AlekSIS

#53173

    ......@@ -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