Re: favicons

On 3/23/20 9:55 PM, Jonathan S. Katz wrote:
> On 3/23/20 9:50 PM, Bruce Momjian wrote:
>> On Thu, Jan 30, 2020 at 02:01:57PM +0100, Daniel Gustafsson wrote:
>>>> On 30 Jan 2020, at 13:09, Marcin Cieslak <saper(at)saper(dot)info> wrote:
>>>>
>>>> On Thu, 30 Jan 2020, Daniel Gustafsson wrote:
>>>>
>>>>>> On 29 Jan 2020, at 16:44, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> wrote:
>>>>>>
>>>>>> www.postgresql.org and commitfest.postgresql.org appear to have different favicons that look just slightly different in a browser tab bar. Perhaps a common one could be found.
>>>>>
>>>>> Nice catch, I hadn't realized but now I can't unsee it. From a quick
>>>>> inspection, the commitfest favicon is the old low-res icon so we should use the
>>>>> pgweb file for the commitfest.
>>>>
>>>> New favicons were generated automatically (the code is available at
>>>> https://repo.or.cz/pgweb-favicon.git ) but I guess it could be improved.
>>>> There was at least one report here that they do not look good in all cases,
>>>> as reported by Peter Geoghegan before.
>>>>
>>>> I'd appreciate some screenshots (maybe real screen photos?) to see what's looking weird.
>>>
>>> The attached screenshot has postgresql.org on the left and the CF app on the
>>> right (in Firefox).
>>
>> Yes, this been fixed? I don't think so.
>
> Someone with commit privs on the commitfest app will have to update it.
> The source of the favicon is in the pgweb tree here:
>
> https://git.postgresql.org/gitweb/?p=pgweb.git;a=blob;f=media/favicon.ico;;hb=HEAD

Attached a patch for convenience.

Jonathan