Re: [pgAdmin4][Patch] - RM 3009 - Right click to copy from data grid,optionally with headers. - Mailing list pgadmin-hackers

From Khushboo Vashi
Subject Re: [pgAdmin4][Patch] - RM 3009 - Right click to copy from data grid,optionally with headers.
Date
Msg-id CAFOhELdqMWrXGhjAWF2FSySw6ZXCOtoDknGETMuHUoN=0jMZUQ@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][Patch] - RM 3009 - Right click to copy from data grid,optionally with headers.  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Responses Re: [pgAdmin4][Patch] - RM 3009 - Right click to copy from data grid,optionally with headers.
List pgadmin-hackers
Hi,

On Mon, Sep 23, 2019 at 3:16 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Khushboo

Following are the review comments:
  • Copy with headers not working when we select a few rows.
I have considered this option only with the entire grid but will fix this.
So, If I select only 2 rows and also select the option Copy with header, then should the header be highlighted? I have highlighted in case of entire grid, with Copy with header option.
@Dave, please suggest.
  • The dropdown should be disabled when the copy button is disabled. 
That is not appropriate. If I have to select the Copy with header option before selecting any row, then not possible with the proposed suggestion. 
  • When user select/de-select "Copy with headers" option dropdown should not be closed. It should be consistent with "Auto Commit/Rollback" or explain options.
  • Feature test "CopySelectedQueryResultsFeatureTest" failed on my machine.
  • Documentation changes are required, update the screenshot wherever applicable. 
Right, I have already thought of that, but when I  checked the current version, I realised that the documentation is not up-to-date with the query-tool toolbar. I didn't find the screen-shots of the many tool-bar dropdown options. So, I would like to involve Abhilasha in this, if everybody agrees, otherwise I can just change the documentation for my patch only.

Thanks,
Khushboo

On Mon, Sep 23, 2019 at 11:42 AM Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi,

Please find the attached patch for RM #3009 - Right click to copy from data grid, optionally with headers.

Query Tool / View data:

Currently the result-set can be copied without header. With this patch the result-set can be copied with the header also and that is optional.

To copy the result-set with header, the option 'Copy with header' is given next with the Copy button in the toolbar in the form of dropdown.

Thanks,
Khushboo


--
Thanks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Private Limited
Mobile: +91 976-788-8246

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: Re: [pgAdmin4][Patch] - RM 3009 - Right click to copy from data grid,optionally with headers.
Next
From: Dave Page
Date:
Subject: Re: [pgAdmin4][Patch] - RM 3009 - Right click to copy from data grid,optionally with headers.