Re: programmers-readme.txt

From: "Dave Page" <dpage(at)vale-housing(dot)co(dot)uk>
To: "Andreas Pflug" <pgadmin(at)pse-consulting(dot)de>, "pgadmin-hackers" <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: programmers-readme.txt
Date: 2006-05-02 14:24:11
Message-ID: E7F85A1B5FF8D44C8A1AF6885BC9A0E4011CA07D@ratbert.vale-housing.co.uk
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

> -----Original Message-----
> From: pgadmin-hackers-owner(at)postgresql(dot)org
> [mailto:pgadmin-hackers-owner(at)postgresql(dot)org] On Behalf Of Dave Page
> Sent: 02 May 2006 10:24
> To: Andreas Pflug; pgadmin-hackers
> Subject: Re: [pgadmin-hackers] programmers-readme.txt
>
>
>
> > -----Original Message-----
> > From: pgadmin-hackers-owner(at)postgresql(dot)org
> > [mailto:pgadmin-hackers-owner(at)postgresql(dot)org] On Behalf Of Dave Page
> > Sent: 02 May 2006 10:22
> > To: Andreas Pflug; pgadmin-hackers
> > Subject: Re: [pgadmin-hackers] programmers-readme.txt
> >
> >
> > 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(!).
>
> Scrub that - it does add it to the menu, but it doesn't work.
> There's only one item added though.

OK, got it working - I hadn't SVN updated to get your fix from yesterday
(sorry).

So currently:

actionFactory works exactly as contextActionFactory should, adding menu
items to the Tools menu (under the appropriate sub menu) and items to
the context menu, either directly if there is only one, or under the
submenu if more than one.

contextActionFactory does all of the above, but also adds additional
copies of each menu item to the root of the context menu so you end up
with:

Contextmenu
-> Submenu
-> Foo
-> Bar
-> Foo
-> Bar

Regards, Dave.

Browse pgadmin-hackers by date

  From Date Subject
Next Message Edward Di Geronimo Jr. 2006-05-02 21:31:24 Re: Paste rows rough patch
Previous Message svn 2006-05-02 14:18:35 SVN Commit by dpage: r5113 - in trunk/pgadmin3/src: frm include main