public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Thomas Lamprecht <t.lamprecht@proxmox.com>
To: Aaron Lauterer <a.lauterer@proxmox.com>,
	Proxmox VE development discussion <pve-devel@lists.proxmox.com>
Subject: Re: [pve-devel] [PATCH futter_frontend 1/2] fix android splash screen logo
Date: Tue, 21 Dec 2021 10:36:46 +0100	[thread overview]
Message-ID: <3443637a-4705-59b9-ed79-8c6c19d3bf8b@proxmox.com> (raw)
In-Reply-To: <c83cb42a-dd97-ff84-fa83-ae5ced14b766@proxmox.com>

On 21/12/2021 10:27, Aaron Lauterer wrote:
> 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.

ack, thanks for clarifying this to me.

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

yeah, could make sense, anyhow if the vector graphic works everywhere I'm glad to 
switch over to that one.




      reply	other threads:[~2021-12-21  9:36 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-20 15:38 Aaron Lauterer
2021-12-20 15:38 ` [pve-devel] [PATCH flutter_frontend 2/2] remove unused drawables for android splash screen Aaron Lauterer
2021-12-20 19:25   ` Thomas Lamprecht
2021-12-20 19:21 ` [pve-devel] [PATCH futter_frontend 1/2] fix android splash screen logo Thomas Lamprecht
2021-12-21  9:25   ` Aaron Lauterer
2021-12-20 19:37 ` Thomas Lamprecht
2021-12-21  9:27   ` Aaron Lauterer
2021-12-21  9:36     ` Thomas Lamprecht [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3443637a-4705-59b9-ed79-8c6c19d3bf8b@proxmox.com \
    --to=t.lamprecht@proxmox.com \
    --cc=a.lauterer@proxmox.com \
    --cc=pve-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal