Message-ID: <46141C4F.3070901@hagander.net>
Date: Wed, 04 Apr 2007 23:44:47 +0200
From: Magnus Hagander <magnus@hagander.net>
User-Agent: Thunderbird 2.0.0.0pre (Windows/20070404)
MIME-Version: 1.0
To: Guillaume Lelarge <guillaume@lelarge.info>
CC: pgsql-www@postgresql.org
Subject: Re: Typos fixes and an accesskey
References: <46118E2E.7000706@lelarge.info>
In-Reply-To: <46118E2E.7000706@lelarge.info>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit

Guillaume Lelarge wrote:
> Hi,
> 
> I send again my patchs because they don't seem to appear on the mailing
> list. I just hope you won't receive the two same messages.
> 
> You'll find attached a .tar.gz which contains two patches.
>
>
> The first one, typos.patch, fixes a few typos I found while translating
> the templates. Please, review them because I'm not a native english
> speaker.

This patch appears to contain a whole lot of changes that are just
whitespace (such as the indention). That makes it much harder to find
out what has really changed. Can you submit it without the whitespace
changes?


> The second one, searchaccesskey.patch, adds an accesskey (q) for the
> search text box field. It's a quick way to access this text box. You can
> change the key if you want but I would really like to see an access key
> on this field.

I'm not really sure about what you can and can not do with accesskeys,
but is there a particular reason you chose "q" for the search feature?
Seems like "s" would be a more logical choice?

//Magnus