Re: programmers-readme.txt - Mailing list pgadmin-hackers

From Dave Page
Subject Re: programmers-readme.txt
Date
Msg-id E7F85A1B5FF8D44C8A1AF6885BC9A0E4011CA057@ratbert.vale-housing.co.uk
Whole thread Raw
In response to programmers-readme.txt  (Andreas Pflug <pgadmin@pse-consulting.de>)
List pgadmin-hackers

> -----Original Message-----
> From: Andreas Pflug [mailto:pgadmin@pse-consulting.de]
> Sent: 30 April 2006 18:59
> To: Dave Page; pgadmin-hackers
> Subject: programmers-readme.txt
>
> Hm, I could have sworn that I already added
> programmers-readme.txt months ago to svn. Anyway, please
> review it if it makes sense to you.

Back at coding now, so...

With this, and the refactoring you've done in frmMain/events.cpp, the
factory stuff is infinitely easier to understand and no longer the
incomprehensible mess that it previously was ( :-p ) so yes, it does
make sense to me!

One thing I did notice - if using contextActionFactory, each item is
added to the context menu twice, the first copy of which doesn't work. A
basic actionFactory *does* add the item to the context menu correctly
though(!).

Regards, Dave.

pgadmin-hackers by date:

Previous
From: "Dave Page"
Date:
Subject: Re: Updated copy patch
Next
From: "Dave Page"
Date:
Subject: Re: programmers-readme.txt