From: Guillaume Lelarge <guillaume@lelarge.info>
To: Dave Page <dpage@pgadmin.org>
Cc: Luis Ochoa <ziul1979@gmail.com>
Cc: pgadmin-hackers <pgadmin-hackers@postgresql.org>
Subject: Re: Database designer
Date: Tue, 06 Mar 2012 09:01:32 +0100
Message-ID: <1331020892.3210.50.camel@localhost.localdomain> (raw)
In-Reply-To: <CA+OCxoxzEWuKC-M5XOjSudyBBKKaYxZg2U=U2jE+0XLOe5O3bQ@mail.gmail.com>
References: <CA+OCxoyNRYDVnR2Lc-c5bx-y+wkFaZ6NVRMZ2v3t0=Gs208P1A@mail.gmail.com>
<1330617986.2275.32.camel@localhost.localdomain>
<CA+OCxoy5yvwzpb_bL8owYFXRJOFq_5rZsyNL3s8w-42sYaOSeA@mail.gmail.com>
<CAFk_LVzz+C9NZ0R1WuhQ26g8xeowdbT9_MJQe7DgJ8Yu9gDWLQ@mail.gmail.com>
<CA+OCxoxzEWuKC-M5XOjSudyBBKKaYxZg2U=U2jE+0XLOe5O3bQ@mail.gmail.com>
On Mon, 2012-03-05 at 15:48 +0000, Dave Page wrote:
> Hi
>
> On Mon, Mar 5, 2012 at 3:29 PM, Luis Ochoa <ziul1979@gmail.com> wrote:
> >
> > Autonaming can be turn off, but the idea of adding this feature arises from
> > good database design practices and the idea of having an initial default
> > name for it, if the user don't want to use, just overwrite it and it
> > automatically turn off.
> > Just to clarify, What's the suggestion for it?
>
> Remove the default, to match the behaviour elsewhere in the
> application. You don't need to provide a default to require the user
> to name the table - that can be required anyway (and would be, if the
> user wants a usable script to be generated). Plus the current default
> really doesn't meet good practices with PostgreSQL, as the default
> name is mixed case.
>
I removed it yesterday evening.
> >>> 7) Why do we have to specify a short table name?
> >>>
> >>
> >> You're not required.
> >
> >> OK, let me rephrase. Why am I asked for one? What effect does it have
> >> on the generated schema?
> >
> >
> > Is for the use of autonaming feature. If that feature is turned off it
> > shouldn't be need any more.
>
> Auto naming of what?
>
FK, and index IIRC.
> > You can doit by two ways: minimize button and dragging table border (size
> > icon) from the button. It sounds like a bug, normal behavior is just like a
> > window with maximize/minimize buttons, I'm not sure about your problem, can
> > you give me the steps to reproduce it, probably is a bug.
>
> I can't find any way to resize using the mouse. I did get it to
> minimise again, when I finally managed to click the _ icon instead of
> +, but cannot get it to maximise again - all that happens is the
> button icon changes.
>
Yeah, it needs to be fixed with a better UI.
> > I really don't believe this feature should be removed, it should fixed I
> > agree, but is a good designed feature, very flexible, able to do it things
> > that other databases designer (even comercial ones) aren't able to do it.
>
> Agreed - but just having has lots of cool features doesn't mean it's
> ready to ship. There are a number of issues that I've identified, and
> probably more I haven't found yet, and we won't ship a feature in
> pgAdmin until we believe it's working to an appropriate standard -
> that means bug free (with some allowance for low-impact issues that
> cannot easily be resolved, and don't materially affect the user), with
> a UI that's usable on all the supported platforms and is consistent
> with the rest of the UX throughout the app, and is written in a way
> that is maintainable moving forwards (the latter, not being an issue
> in this case from what I've seen).
>
On such a huge patch, it's really hard to work without commiting some
parts of it. So I decided to commit each milestone of the project. I
agree with you that it lacks lots of tweaking, but it's not that bad.
--
Guillaume
http://blog.guillaume.lelarge.infohttp://www.dalibo.com
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: pgadmin-hackers@postgresql.org
Cc: guillaume@lelarge.info, dpage@pgadmin.org, ziul1979@gmail.com
Subject: Re: Database designer
In-Reply-To: <1331020892.3210.50.camel@localhost.localdomain>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox