Re: Fixed one minor issue related to "Reassign objects to"

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

In response to

Responses

Browse pgadmin-hackers by date

  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