From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by lists.proxmox.com (Postfix) with ESMTPS id 8380085C98 for ; Tue, 21 Dec 2021 10:27:36 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id 70EC627341 for ; Tue, 21 Dec 2021 10:27:36 +0100 (CET) Received: from proxmox-new.maurer-it.com (proxmox-new.maurer-it.com [94.136.29.106]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by firstgate.proxmox.com (Proxmox) with ESMTPS id 96AF727333 for ; Tue, 21 Dec 2021 10:27:35 +0100 (CET) Received: from proxmox-new.maurer-it.com (localhost.localdomain [127.0.0.1]) by proxmox-new.maurer-it.com (Proxmox) with ESMTP id 4ABF445220 for ; Tue, 21 Dec 2021 10:27:35 +0100 (CET) Message-ID: Date: Tue, 21 Dec 2021 10:27:34 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.4.0 Content-Language: en-US To: Thomas Lamprecht , Proxmox VE development discussion References: <20211220153839.537253-1-a.lauterer@proxmox.com> <3f42d312-9021-5d83-6e8b-92c07dcb10d8@proxmox.com> From: Aaron Lauterer In-Reply-To: <3f42d312-9021-5d83-6e8b-92c07dcb10d8@proxmox.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-SPAM-LEVEL: Spam detection results: 0 AWL 1.016 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment NICE_REPLY_A -2.012 Looks like a legit reply (A) SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record URIBL_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to URIBL was blocked. See http://wiki.apache.org/spamassassin/DnsBlocklists#dnsbl-block for more information. [android.com] Subject: Re: [pve-devel] [PATCH futter_frontend 1/2] fix android splash screen logo X-BeenThere: pve-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox VE development discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 21 Dec 2021 09:27:36 -0000 On 12/20/21 20:37, Thomas Lamprecht wrote: > On 20/12/2021 16:38, Aaron Lauterer wrote: >> The new splash screen that came with Android 12 has stricter rules >> regarding the size and form of the logo in the splash screen [0]. > > oh, do you have an actual reference to the strict rules you found, as [0] > > As I read [0] quite closely and basically all mentions regarding the > drawable are IMO the opposite of strict, e.g.: > >> The app icon (1) should be a vector drawable, and it can be static or animated. > > And all other references are similarly vague and imply multiple times that > any drawable should work. I think the following line below figure 2 is the important one: "As with adaptive icons, one-third of the foreground is masked (3)." Hinting at the #3 in the figure, which is the area outside the circle. Taking a closer look at the current splash screen, one will see that the drawable is masked in a circular shape. Depending on how it is scaling, the circular mask will be easier or harder to spot. > >> >> We do need to have a drawable that is a square with the logo / icon in >> the center. That means the current approach with the png image resulted >> in it being scaled to the full width and the circular mask cut if off. > > hmm, the bitmap used sets `android:gravity` to `center` though, which is > documented as: > >> Place the object in the center of its container in both the vertical and >> horizontal axis, not changing its size. > > -- https://developer.android.com/guide/topics/resources/drawable-resource#XmlBitmap That probably explains why it did look quite different on your phone than on mine regarding how large and cutoff it appeared, depending on how they are set up regarding the px to dp (device independent pixel) conversion. > >> The result was a broken looking splash screen. > > FWIW, I only run into the following issue: > https://issuetracker.google.com/issues/207095799#comment10 > >> >> I opted for the stacked variant of the logo and saved it as a vector >> drawable in the xml format that android wants. >> >> [0] https://developer.android.com/guide/topics/ui/splash-screen >> >> Signed-off-by: Aaron Lauterer >> --- >> .../main/res/drawable/proxmox_splash_logo.xml | 69 +++++++++++++++++++ >> .../app/src/main/res/values-night/styles.xml | 2 +- >> android/app/src/main/res/values/styles.xml | 2 +- >> 3 files changed, 71 insertions(+), 2 deletions(-) >> create mode 100644 android/app/src/main/res/drawable/proxmox_splash_logo.xml >> >