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

Поиск
Список
Период
Сортировка
От Akshay Joshi
Тема Re: [pgAdmin4][Patch] - RM 3009 - Right click to copy from data grid, optionally with headers.
Дата
Msg-id CANxoLDc=_xWnLLbKQkazXPe+jyZ5eJMs3hZP2W8rC5HobFVGJA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin4][Patch] - RM 3009 - Right click to copy from data grid,optionally with headers.  (Khushboo Vashi <khushboo.vashi@enterprisedb.com>)
Ответы Re: [pgAdmin4][Patch] - RM 3009 - Right click to copy from data grid, optionally with headers.  (Khushboo Vashi <khushboo.vashi@enterprisedb.com>)
Список pgadmin-hackers
Hi Khushboo

Found one minor issue "Header row should get unhighlighted when we deselect 'Copy with headers' and select some new row or deselect any existing selected row."
Screenshot 2019-09-26 at 2.32.43 PM.png

 Apart from that code looks good to me.

On Thu, Sep 26, 2019 at 10:51 AM Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi,

Please find the attached updated patch.

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.
Fixed 
  • The dropdown should be disabled when the copy button is disabled.
  • 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.
Fixed 
  • Feature test "CopySelectedQueryResultsFeatureTest" failed on my machine.
Fixed.

Thanks,
Khushboo 
  • Documentation changes are required, update the screenshot wherever applicable. 

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


--
Thanks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Private Limited
Mobile: +91 976-788-8246
Вложения

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

Предыдущее
От: Khushboo Vashi
Дата:
Сообщение: Re: [pgAdmin4][Patch] - RM 3009 - Right click to copy from data grid,optionally with headers.
Следующее
От: Khushboo Vashi
Дата:
Сообщение: Re: [pgAdmin][RM4701] Webpack optimization