From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Dhiraj Chawla <dhiraj(dot)chawla(at)enterprisedb(dot)com> |
Cc: | Michael Shapiro <mshapiro51(at)gmail(dot)com>, Kieran <kieran(dot)mccusker(at)kwest(dot)co>, "pgadmin-support(at)postgresql(dot)org" <pgadmin-support(at)postgresql(dot)org> |
Subject: | Re: Copy and paste in Edit Data |
Date: | 2013-05-21 15:53:57 |
Message-ID: | CA+OCxow55OmS-RMfZDC+wAFwBcky++wQfB3=JknkoS3FMGyFvw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-support |
Hi
On Tue, May 21, 2013 at 11:14 AM, Dhiraj Chawla
<dhiraj(dot)chawla(at)enterprisedb(dot)com> wrote:
> Hi Dave,
>
> I had some problems with the applying the patch because it seems corrupted
> with characters ^[[m at the beginning and end or lines of the patch code.
> Once I removed them the patch applied properly.
Hmm, must have been where I was working on Windows.
> The patch is working as expected and I have tested in all scenarios that I
> could think of. While reviewing I have only one doubt:
Cool.
> Should'nt we remove the definition and declaration of the OnCopy() function
> from ctlSQLGrid [ void ctlSQLGrid::OnCopy(wxCommandEvent &ev) ] as it is no
> longer being called, owning to the removal of the event trigger in the
> patch?
>
> - EVT_MENU(MNU_COPY, ctlSQLGrid::OnCopy)
No, because ctlSQLGrid::OnCopy is still called by frmQuery to handle
copying from the Query Tool.
> Apart from this the code looks good to me.
Thanks - will apply.
>
> On Tue, May 21, 2013 at 4:26 AM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>
>>
>> On Friday, May 17, 2013, Michael Shapiro wrote:
>>>
>>> I am running 1.16.1 (Windows 7). I can reproduce this. It makes a
>>> difference, though, if you use ctrl-c vs the right-click/Copy. ctrl-c does
>>> the whole entry, right-click/Copy gets just what is highlighted. ctrl-c gets
>>> the whole entry even if nothing is highlighted.
>>>
>>>
>>> On Fri, May 17, 2013 at 5:34 AM, Kieran <kieran(dot)mccusker(at)kwest(dot)co> wrote:
>>>>
>>>> Hi
>>>>
>>>> Sorry if this has been raised before, but it's been a nuisance to me for
>>>> quite a while and I thought I should mention it.
>>>>
>>>> To reproduce the problem :-
>>>>
>>>> 1. Open the Edit Data window on a table
>>>> 2. Click into a text field containing some text and highlight a single
>>>> word
>>>> 3. Paste what you have just copied.
>>>>
>>>> You will get the whole contents of the field not what you just
>>>> highlighted.
>>>>
>>>> My environment is
>>>> Windows Vista
>>>> pgAdmin 1.16.1
>>>>
>>
>> I believe the attached patch fixes this. With it, if you're in Edit Mode
>> in a cell, then Copy works more like it would in a regular text box - only
>> the highlighted text is copied, and there are no quotes etc. added. In View
>> Mode, whatever cells are selected will be copied, and quoting/delimiting
>> applied.
>>
>> Can someone review it please (Dhiraj :-p )?
>>
>> Thanks.
>>
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>
>
>
> --
> regards,
>
> Dhiraj Chawla
> Software Engineer
> EnterpriseDB Corporation
> The Enterprise PostgreSQL Company
>
> Phone: +91-20-30589522
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | David Burton | 2013-05-23 14:15:53 | pgAdmin3 restore getting password authentication failed for user "postgres" |
Previous Message | Dhiraj Chawla | 2013-05-21 11:14:33 | Re: Copy and paste in Edit Data |