Re: Fwd: Filter by Selection on Grid

From: "Robins Tharakan" <tharakan(at)gmail(dot)com>
To: "Dave Page" <dpage(at)postgresql(dot)org>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: Fwd: Filter by Selection on Grid
Date: 2008-02-05 17:57:45
Message-ID: 36af4bed0802050957o63f4673fh8e1f080fa50b34fd@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

> Hi Dave,
>
> > OK, slow down :-). ... snip If you want to add more functionality, post
> a new patch,
> > otherwise the process just becomes a lot more complicated.
>
> Sorry for jumping the gun !
> I didn't think that this kind of a complication could arise....Guess it
> shows that this is my first patch submission :-)
>
> Sure, I'll try and keep separate patches in separate emails.
> Attached is reworked patch which takes care of the issues that you
> mentioned.
>
> Updates in this patch:
> 1. Removed the extra separator that came up on right-click.
> 2. Enabled / Disabled the menu options while a query is running
> 3. The code also disables the right-click while a thread->IsRunning() is
> true. (That should effectively disable the right-click when not needed right
> ?)
>
> > One hint - test in a debug enabled build as you should then see
> > asserts for things like enabling non-existant menu options.
>
> I didn't see any error messages, but that could be probably because I
> corrected the code before I ran it in debug mode.
> Probably this'll help sometime in the future.
>
> Anything else that needs more work ?
>
> *Robins*
>

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message svn 2008-02-05 18:06:39 SVN Commit by mha: r7055 - trunk/pgadmin3/pgadmin/frm
Previous Message Robins Tharakan 2008-02-05 17:57:18 Re: Fwd: Filter by Selection on Grid