Re: Patch: Config editors

From: "Dave Page" <dpage(at)vale-housing(dot)co(dot)uk>
To: "Magnus Hagander" <mha(at)sollentuna(dot)net>, <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Patch: Config editors
Date: 2006-01-25 09:44:42
Message-ID: E7F85A1B5FF8D44C8A1AF6885BC9A0E40103DAA8@ratbert.vale-housing.co.uk
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Thanks, patch applied. I made some additional changes to the editors to
prevent undo on the wrong line, and to prevent deletion of the blank
'new' item.

Regards, Dave.

> -----Original Message-----
> From: Magnus Hagander [mailto:mha(at)sollentuna(dot)net]
> Sent: 23 January 2006 15:41
> To: Dave Page; pgadmin-hackers(at)postgresql(dot)org
> Subject: RE: [pgadmin-hackers] Patch: Config editors
>
> Updated version attached:
>
> * Include xrc file that was missing last time ;-) Haven't included the
> appropriate .cpp update because it was huge ;-)
> * Fix help button in all editors (didn't work in hba editor either..)
> * Add delete button
> * Disable the hint button in the hba and pgpass editors, because we
> don't hint anyway
> * "Fix" the undo by disabling the undo button when you delete
> something,
> so you don't think it works (see below)
>
>
> Now, as for the undo functionality. The undo as it is now in these
> dialogs don't work like a standard undo which is "undo the
> latest edit".
> It is "undo the latest edit on the current line". So this
> patch actually
> follows that (since there is no way to select a deleted
> line). Changing
> this to a "normal undo" is a different patch alltogether ;-)
>
> For example:
> 1) Edit line #1
> 2) Edit line #2
> 3) Select line #1
> 4) Click undo
> 5) Watch how the change from (1) gets reverted, not the one from (2)
>
>
> //Magnus
>
>
> > -----Original Message-----
> > From: Dave Page [mailto:dpage(at)vale-housing(dot)co(dot)uk]
> > Sent: Monday, January 23, 2006 4:00 PM
> > To: Magnus Hagander; pgadmin-hackers(at)postgresql(dot)org
> > Subject: RE: [pgadmin-hackers] Patch: Config editors
> >
> > Patch discussed with Magnus on IM. To be reposted with fixes for the
> > following:
> >
> > - Undo button doesn't undo a delete.
> > - There is no delete button on the dialogue
> > - Help button on the main editor window doesn't work.
> >
> > Regards, Dave.
> >
> > > -----Original Message-----
> > > From: pgadmin-hackers-owner(at)postgresql(dot)org
> > > [mailto:pgadmin-hackers-owner(at)postgresql(dot)org] On Behalf Of Magnus
> > > Hagander
> > > Sent: 23 January 2006 13:18
> > > To: pgadmin-hackers(at)postgresql(dot)org
> > > Subject: [pgadmin-hackers] Patch: Config editors
> > >
> > > Attached patch implements an editor for pgpass.conf/.pgpass
> > modeled on
> > > the HBA editor. In doing so it also:
> > >
> > > * Adds the ability to delete a row to the pg_hba editor
> > > * Moves the "get the name of the password file" to
> pgSettings so it
> > > can be shared. Sets up a tiny bit of infrastructure for
> > when there may
> > > be more config files.
> > >
> > > Trying a .zip this time so it doesn't get eaten by the
> list filter.
> > >
> > > //Ma <<Config_editors.zip>> gnus
> > >
> > >
> > >
> > >
> >
>

Browse pgadmin-hackers by date

  From Date Subject
Next Message Magnus Hagander 2006-01-25 09:44:49 Re: Patch: Config editors
Previous Message svn 2006-01-25 08:37:27 SVN Commit by dpage: r4958 - branches/REL-1_4_0_PATCHES/pgadmin3/src/frm