From: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Fixed one minor issue related to "Reassign objects to" |
Date: | 2013-08-20 07:19:42 |
Message-ID: | CANxoLDfd0wy0-KFANuQE3U+KVxjop7Db19HZQUAN7Rj=e65PGg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Mon, Aug 5, 2013 at 5:29 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
> Hi
>
> On Fri, Aug 2, 2013 at 8:53 AM, Akshay Joshi
> <akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
> > Hi Dave
> >
> > I have fixed one minor issue when user right click on any login role and
> > select "Reassign/Drop Owned..." context menu. When user click on "OK"
> > button, but there is no role in the combo box. Attached is the
> screenshot of
> > the error.
> >
> > I have fixed the issue by disabling the "OK" button if there is no role
> in
> > the combo box. Attached is the patch file, please review it and if it
> looks
> > good then can you please commit it.
>
> Thanks - committed. I notice that dialogue needs some additional work
> - it looks like it was thrown together with very little thought :-(.
> Can you also please:
>
> - Ensure the default position is center-owner.
>
> - Ensure the position is saved and restored after the first usage.
>
> - Change to a fixed-size.
>
Fixed. Patch file attached.
>
> Thanks Akshay.
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
--
*Akshay Joshi
Senior Software Engineer
EnterpriseDB Corporation
The Enterprise PostgreSQL Company
Phone: +91 20-3058-9522
Mobile: +91 976-788-8246*
Attachment | Content-Type | Size |
---|---|---|
ReassignObject_dialog_fixes.patch | application/octet-stream | 1.4 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2013-08-20 07:30:34 | pgAdmin III commit: Ensure VALID UNTIL isn't inadvertently set to the e |
Previous Message | Ashesh Vashi | 2013-08-20 07:15:59 | Re: Bug in the graphical explain |