Re: Discussion - Search Objects

From: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
To: Jasmin Dizdarevic <jasmin(dot)dizdarevic(at)gmail(dot)com>
Cc: Dave Page <dpage(at)pgadmin(dot)org>, pgadmin-hackers(at)postgresql(dot)org
Subject: Re: Discussion - Search Objects
Date: 2011-06-15 21:38:55
Message-ID: 1308173936.1993.90.camel@laptop
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Wed, 2011-06-15 at 08:08 +0200, Guillaume Lelarge wrote:
> On Wed, 2011-06-15 at 01:03 +0200, Jasmin Dizdarevic wrote:
> > Hi,
> >
> >
> > please take another look on it. The UI should work correctly now.
> >
> >
> > Linux make files should also be correct now.
> >
>
> It works much better now. You still have one issue with primary key (and
> I guess, unique constraint and exclusive constraints too). You show them
> as constraints and indexes, whereas pgAdmin only shows them as
> constraints. You shouldn't display indexes for constraints with implicit
> indexes.
>
> And I didn't look at the code yet.
>

Seems mostly good. The only question I have is: why did you choose to
name it frm* instead of dlg*? it behaves as a dialog, not as a form.

--
Guillaume
http://blog.guillaume.lelarge.info
http://www.dalibo.com

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Jasmin Dizdarevic 2011-06-15 21:48:26 Re: Discussion - Search Objects
Previous Message Guillaume Lelarge 2011-06-15 21:30:05 Re: Ownership of a new view