Re: Event Triggers are not listed in the browser view options on the Settings dialogue

Поиск
Список
Период
Сортировка
От Dhiraj Chawla
Тема Re: Event Triggers are not listed in the browser view options on the Settings dialogue
Дата
Msg-id CAJgtxT46kXwzTURtcU7b-aNBNDd4F_TCvea3K8K1K6XzJxPtZA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Event Triggers are not listed in the browser view options on the Settings dialogue  (Dave Page <dpage@pgadmin.org>)
Ответы Re: Event Triggers are not listed in the browser view options on the Settings dialogue
Список pgadmin-hackers
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@pgadmin.org> wrote:
Hi

On Fri, Sep 20, 2013 at 1:13 PM, Dhiraj Chawla
<dhiraj.chawla@enterprisedb.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

Вложения

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Dave Page
Дата:
Сообщение: Re: Event Triggers are not listed in the browser view options on the Settings dialogue
Следующее
От: Neel Patel
Дата:
Сообщение: Slony Issue Fix in pgAdmin