public inbox for [email protected]  

help / color / mirror / Atom feed

From: Magnus Hagander <[email protected]>
To: Bruce Momjian <[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 23:46:31 +0100
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>

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
> 

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