From: | John Obaterspok <john(dot)obaterspok(at)gmail(dot)com> |
---|---|
To: | Sanket Mehta <sanket(dot)mehta(at)enterprisedb(dot)com> |
Cc: | Dave Page <dpage(at)pgadmin(dot)org>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [PATCH] Add Commit/Rollback toolbar action |
Date: | 2015-06-17 12:22:03 |
Message-ID: | CAOscVdLeZHJ=Sd5zMgwHodVdkh5hXj9Ex9hZ7AijXaOX8tH=3Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Thanks,
I can try to look at disabling when not in transaction mode + the short
cuts.
Regarding the icons, toad for oracle uses a database icon which the arrows
point to/from. Perhaps we should add something *similar*?
[image: Infogad bild 1]
-- john
2015-06-17 13:44 GMT+02:00 Sanket Mehta <sanket(dot)mehta(at)enterprisedb(dot)com>:
> 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 | Sanket Mehta | 2015-06-18 09:24:59 | Re: [PATCH] Fix crash when disabling auto commit |
Previous Message | Sanket Mehta | 2015-06-17 11:44:22 | Re: [PATCH] Add Commit/Rollback toolbar action |