Re: favicons

From: Dave Page <dpage(at)pgadmin(dot)org>
To: "Jonathan S(dot) Katz" <jkatz(at)postgresql(dot)org>
Cc: Bruce Momjian <bruce(at)momjian(dot)us>, Daniel Gustafsson <daniel(at)yesql(dot)se>, Marcin Cieslak <saper(at)saper(dot)info>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, Peter Geoghegan <pg(at)bowt(dot)ie>, PostgreSQL WWW <pgsql-www(at)postgresql(dot)org>
Subject: Re: favicons
Date: 2020-03-24 09:14:29
Message-ID: CA+OCxowWtFxavGYJJA7f6Yg+qXm7jnpJNSky0pJD7_hh=6_47g@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-www

On Tue, Mar 24, 2020 at 1:59 AM Jonathan S. Katz <jkatz(at)postgresql(dot)org>
wrote:

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

Thanks, committed.

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Browse pgsql-www by date

  From Date Subject
Next Message MichaelDBA 2020-03-24 13:29:42 Wiki editor request
Previous Message Daniel Gustafsson 2020-03-24 07:15:46 Re: favicons