Re: PATCH: Column Level Privileges - Mailing list pgadmin-hackers

From Dave Page
Subject Re: PATCH: Column Level Privileges
Date
Msg-id 937d27e10902020533k532f4ee5i1eb35936e97afcfa@mail.gmail.com
Whole thread Raw
In response to Re: PATCH: Column Level Privileges  (Ashesh Vashi <ashesh.vashi@enterprisedb.com>)
Responses Re: PATCH: Column Level Privileges
List pgadmin-hackers
Hi

On Mon, Feb 2, 2009 at 12:55 PM, Ashesh Vashi
<ashesh.vashi@enterprisedb.com> wrote:
> Hi Dave,
>
> Dave Page wrote:
>
> - If the dialog is the default (minimum) size when opened, the
> privileges pane does not size correctly. Observed on Windows.
>
> I could not reproduce it on windows (Vista). :(
> Could you please send a screen shot for it?

Attached before and after resize screenshots (it fixes itself if you
resize the dialogue)

> And also found one problem, when privileges are for group, "group " was not
> appended in privileges lists (ctlSecurityPanel). I found the same problem on
> existing privileges dialogs too. :(

Thats intentional - we dropped the 'group' keyword when roles were
introduced. It looks like your patch fixes a related issue with the
incorrect icon being used - we should keep that part of the fix :-)


--
Dave Page
EnterpriseDB UK:   http://www.enterprisedb.com

Attachment

pgadmin-hackers by date:

Previous
From: Ashesh Vashi
Date:
Subject: Re: PATCH: Column Level Privileges
Next
From: svn@pgadmin.org
Date:
Subject: SVN Commit by dpage: r7576 - trunk/pgadmin3/pgadmin/ui