From: | Dhiraj Chawla <dhiraj(dot)chawla(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Event Triggers are not listed in the browser view options on the Settings dialogue |
Date: | 2013-09-26 09:07:57 |
Message-ID: | CAJgtxT4XCCkDtEaB_wXAKtRAuXLmzwuPxH2UOUz5K4PXtDxy5A@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Thanks Dave.
regards,
*Dhiraj Chawla*
Senior Software Engineer
EnterpriseDB Corporation
The Enterprise PostgreSQL Company
Phone: +91-20-30589522
On Thu, Sep 26, 2013 at 2:37 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
> Thanks, applied for 1.18.1+.
>
> On Tue, Sep 24, 2013 at 10:39 AM, Dhiraj Chawla
> <dhiraj(dot)chawla(at)enterprisedb(dot)com> wrote:
> > Hi Dave,
> >
> > Please find the updated patch. I have moved the Event Triggers node
> before
> > the Extensions node in the treeview. I have also re-arranged the the
> display
> > options list to the best of my knowledge. Please review the patch and
> let me
> > know your thoughts on the same.
> >
> > regards,
> >
> > Dhiraj Chawla
> > Senior Software Engineer
> > EnterpriseDB Corporation
> > The Enterprise PostgreSQL Company
> >
> > Phone: +91-20-30589522
> >
> >
> > On Mon, Sep 23, 2013 at 9:08 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
> >>
> >> Hi
> >>
> >> On Fri, Sep 20, 2013 at 1:13 PM, Dhiraj Chawla
> >> <dhiraj(dot)chawla(at)enterprisedb(dot)com> wrote:
> >> > Hi Dave,
> >> >
> >> > The Event Triggers options is not listed in the browser view options
> on
> >> > the
> >> > Settings dialog of pgAdmin, which would allow a user the hide/show the
> >> > Event
> >> > Triggers collection node. Attached is the patch that fixes this issue.
> >> >
> >> > Let me know your view on the same.
> >>
> >> Thanks, that works. However, I notice it's not in the right place. We
> >> normally display the nodes alphabetically - so can you please move the
> >> Event Triggers node in the treeview so it appears before Extensions
> >> (which I believe would be correct)?
> >>
> >> Then, the position in the list on frmOptions should be corrected to
> >> match. We list items there in the order they'd be seen in the
> >> treeview, running down and across the logical hierarchy - which I
> >> think means both Extensions and Event Triggers are in the wrong place
> >> - they should probably be between Casts and Languages (though, looking
> >> more closely, I think all the Foreign Table stuff is in the wrong
> >> place too, so feel free to fix that as well)!
> >>
> >> Thanks!
> >>
> >> --
> >> Dave Page
> >> Blog: http://pgsnake.blogspot.com
> >> Twitter: @pgsnake
> >>
> >> EnterpriseDB UK: http://www.enterprisedb.com
> >> The Enterprise PostgreSQL Company
> >
> >
>
>
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2013-09-26 09:11:54 | pgAdmin III commit: Ensure DDLScript will work with Slony 2.2+ which ha |
Previous Message | Dave Page | 2013-09-26 09:07:14 | Re: Event Triggers are not listed in the browser view options on the Settings dialogue |