Skip site navigation (1) Skip section navigation (2)

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-04 18:36:26
Message-ID: (view raw, whole thread or download thread mbox)
Lists: pgadmin-hackers
Hi Dave,

Thanks for the feedback. A positive note certainly helps :)

I've tried to work on the things mentioned and although I can think of a few
more additions, I think I'd work on them separately once this gets through.

A few updates:
1. Replaced all SQL related _() with wxT()
2. Created a separate Tools Menu. Moved 'Sort / Filter ...' there and added
all the new 'right-clickable options' there as well.
3. Right-clicking a cell makes that cell the selected cell on the grid.
4. Empty strings are now taken care of as well.
5. As earlier, all options(including the sort options) refresh the grid

Hope I didn't miss anything!

*Robins Tharakan*

---------- Forwarded message ----------
From: Dave Page <dpage(at)postgresql(dot)org>
Date: Feb 4, 2008 5:55 PM
Subject: Re: [pgadmin-hackers] Fwd: Filter by Selection on Grid
To: robins(at)pobox(dot)com
Cc: pgadmin-hackers(at)postgresql(dot)org


On Feb 1, 2008 4:38 AM, Robins Tharakan <tharakan(at)gmail(dot)com> wrote:
> The good part is that now the patch does the following:
> 1. Provide options for 'Filter by Selection' / 'Filter Excluding
> / 'Remove Filter'
>  2. Takes care if cells have NULL value
> 3. Takes multiple Include / Exclude selections and refreshes automatically

Looks good :-)

Some minor comments:

- the _() macro is used for strings that need translation, so please
use wxT() for those that don't, such as parts of SQL queries.

- I think we should have the menu options on a new Tools menu as well
(we normally duplicate context menu options onto the main menu for
accessibility reasons). We should probably move the Sort/Filter option
onto there as well.

> Limitations that I plan to remove soon:
> 1. The cell chosen is the 'Selected cell' and that may not be the one
> right-clicked. I'd change that soon.

Yeah - I've actually been thinking that for various reasons we might
want to have the active cell follow the mouse clicks and row/column
selections anyway.

>  2. I am unable to evaluate strings if they are Empty, so
> including/excluding empty-cells (not NULLs) doesn't do anything !
> Frustrating ! May be I just need a break, a bit of reading and I'll get
> done soon as well!

Don't forget that empty strings are displayed as '' (two single
quotes), and a string that actually is two single quotes is \'\'.
You'll need to allow for those when building your filters.

> In case this patch is considered by and large fine, and in case you are ok
> with more additions, I'll try adding more features in the right click menu
> (Filter by Custom String / Sort options etc..) similar to what we have in
> the Access UI.

I look forward to it :-)

Thanks, Dave

Attachment: filter_by_selection_v9.patch
Description: text/x-patch (10.8 KB)

In response to


pgadmin-hackers by date

Next:From: ErikDate: 2008-02-04 21:51:04
Subject: pgadmin using pg_dump 8.3beta2
Previous:From: Robins TharakanDate: 2008-02-04 18:32:43
Subject: Re: Fwd: Filter by Selection on Grid

Privacy Policy | About PostgreSQL
Copyright © 1996-2017 The PostgreSQL Global Development Group