public inbox for [email protected]  
help / color / mirror / Atom feed
From: 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 18:02:22 -0500 (EST)
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>


I think the feedback was that using the standard colors was too likely
to look bad on the web site.  Perhaps you should just increase the
different between the blues on your version and commit it, and then ask
for more feedback.

---------------------------------------------------------------------------

Magnus Hagander wrote:
> I was hoping someone else would comment on which one of the two they
> preferred before we committed it. And then I promptly forgot about it ;-)
> 
> So, which one do people prefer?
> 
> //Magnus
> 
> 
> Bruce Momjian wrote:
> > 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