From: | Sanket Mehta <sanket(dot)mehta(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | John Obaterspok <john(dot)obaterspok(at)gmail(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [PATCH] Add Commit/Rollback toolbar action |
Date: | 2015-06-17 11:44:22 |
Message-ID: | CA+yw=mOmxMxxWDwKgzf7yQNz5ZLmVm25HiqKTHR-Z6iX6CnXcA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi John,
I have applied the patch and below are my suggestions:
1. Rollback and commit icons should be disabled session is not in
transaction mode
2. Rollback and Commit should also be included in Query menu and individual
short cut should be assigned to them.
Dave,
Can we have any other icons for both the functionality as these icons looks
odd to me.
Regards,
Sanket Mehta
Sr Software engineer
Enterprisedb
On Tue, Jun 16, 2015 at 5:03 PM, Sanket Mehta <sanket(dot)mehta(at)enterprisedb(dot)com
> wrote:
> Sure Dave.
>
> Regards,
> Sanket Mehta
> Sr Software engineer
> Enterprisedb
>
> On Tue, Jun 16, 2015 at 4:55 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Sanket, can you look at this please?
>>
>> Sent from my iPad
>>
>> > On 15 Jun 2015, at 16:33, John Obaterspok <john(dot)obaterspok(at)gmail(dot)com>
>> wrote:
>> >
>> > Hello,
>> >
>> > With the newly added option to disable auto commit (hurray) I really
>> missed commit/rollback toolbar buttons. Attached patch adds this.
>> >
>> > -- john
>> >
>> > <0001-Add-Commit-Rollback-toolbar-action.patch>
>> >
>> > --
>> > Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
>> > To make changes to your subscription:
>> > http://www.postgresql.org/mailpref/pgadmin-hackers
>>
>
>
From | Date | Subject | |
---|---|---|---|
Next Message | John Obaterspok | 2015-06-17 12:22:03 | Re: [PATCH] Add Commit/Rollback toolbar action |
Previous Message | Sanket Mehta | 2015-06-16 11:33:10 | Re: [PATCH] Add Commit/Rollback toolbar action |