public inbox for [email protected]
help / color / mirror / Atom feedFrom: Bruce Momjian <[email protected]>
To: Magnus Hagander <[email protected]>
Cc: Robert Treat <[email protected]>
Cc: Tom Lane <[email protected]>
Cc: [email protected]
Cc: [email protected]
Subject: Re: Patch for text.css
Date: Sat, 20 Jan 2007 17:13:03 -0500 (EST)
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
Are we going to apply this?
---------------------------------------------------------------------------
Magnus Hagander wrote:
> Robert Treat wrote:
> > On Monday 08 January 2007 11:20, Magnus Hagander wrote:
> >> On Sun, Jan 07, 2007 at 08:53:30PM -0500, Tom Lane wrote:
> >>> Magnus Hagander <[email protected]> writes:
> >>>> Or should we perhaps just stop setting a color at all on it? That would
> >>>> let the browser choose color?
> >>> I wouldn't vote for that; the odds are good that the browser's defaults
> >>> would look ugly, or maybe even be nearly invisible against our
> >>> backgrounds. We *should* set the link colors ... just not to the same
> >>> thing.
> >> I've applied the patch to http://magnus-master.pgadmin.org with the
> >> color as suggested, for review.
> >>
> >> Thoughts? Different color? Suggestions?
> >>
> >
> > I had a hard time differentiating it last night, so I changed it to use the
> > standard color on http://xzilla.postgresql.org just to get a different look.
> > Anyone interested in comparing can check it out.
>
> I personally think yours is clearer, but I'm a bit concerned on the
> third opinion ;-) Anybody else care to chime in?
>
> //Magnus
>
> ---------------------------(end of broadcast)---------------------------
> TIP 9: In versions below 8.0, the planner will ignore your desire to
> choose an index scan if your joining column's datatypes do not
> match
--
Bruce Momjian [email protected]
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
view thread (21+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected], [email protected]
Subject: Re: Patch for text.css
In-Reply-To: <[email protected]>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox