Re: Ticket 119: handling opclass in dlgIndex - Mailing list pgadmin-hackers

From Guillaume Lelarge
Subject Re: Ticket 119: handling opclass in dlgIndex
Date
Msg-id 4B37323F.6040304@lelarge.info
Whole thread Raw
In response to Re: Ticket 119: handling opclass in dlgIndex  (Dave Page <dpage@pgadmin.org>)
List pgadmin-hackers
Le 23/12/2009 20:04, Dave Page a écrit :
> On Wed, Dec 23, 2009 at 6:08 PM, Guillaume Lelarge
> <guillaume@lelarge.info> wrote:
>
>> I don't really know what the english standard way of casing words is. I
>> know that, in French, you're supposed to put the first character of a
>> string in uppercase. In English, I suppose I need to put the first
>> character of each word in uppercase. So, I think we need to upper case
>> the n in "Column name" (ie, "Column Name"). Can you confirm?
>
> Sorry - I meant to explain more but was multi-tasking and messed up :-(
>

No problem :)

> In grammatically correct English, only the first word of a sentence
> should be capitalised, but looking at other software like XCode and
> Firefox, it seems it's not that simple. In fact, as I'm trying to
> write this, I'm struggling to see a pattern in either of those apps -
> for example, in XCode's preferences pane I see options like:
>
> Automatically Suggest
> Suggestion delay
> Default File Encoding
> Tab indents
>
> So, I think we should use English rules, and only capitalise the first
> word, *except* for following words which have special meaning because
> they are SQL keywords for example, or acronyms.
>

OK. I've done that on the dlgIndex source (.cpp and .xrc files). We need
to do this on the complete source.

New patch attached.

>> Will do. Should it be in another patch?
>
> Probably better, if only to keep the commit logs focused on one change.
>

OK. I'll do in another patch, but I would like to commit this one before.

>> Or we have to tell them we need to drop and create their index when they
>> click the OK button, perhaps giving them the choice to do it
>> concurrently. I wonder if there are other objects that would need that
>> treatment.
>
> I can't say I'm that excited about that. It's probably safer for them
> to create the new index manually and then drop the old. I suppose we
> could create, drop, rename, but there's still something about the idea
> that gives me the heebie-jeebies.
>

OK, not really a big issue anyways.


--
Guillaume.
 http://www.postgresqlfr.org
 http://dalibo.com

Attachment

pgadmin-hackers by date:

Previous
From: Guillaume Lelarge
Date:
Subject: Re: Maintenance window review
Next
From: Mariano Reingart
Date:
Subject: Re: Spanish (es_ES) translation