Re: [pgAdmin][RM5488] Tooltip information does not display properly if user check all options under explain analyze

Поиск
Список
Период
Сортировка
От Aditya Toshniwal
Тема Re: [pgAdmin][RM5488] Tooltip information does not display properly if user check all options under explain analyze
Дата
Msg-id CAM9w-_==zEUfHD63TpKbBAMUxNxLwOO01LtiNLSrT9M8oL-eDw@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin][RM5488] Tooltip information does not display properly if user check all options under explain analyze  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Ответы Re: [pgAdmin][RM5488] Tooltip information does not display properly if user check all options under explain analyze  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Список pgadmin-hackers
Дерево обсуждения
Hi Akshay,

Attached is the updated patch. Please review.

On Tue, Jan 12, 2021 at 2:51 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Aditya

Code looks good to me. Below are the review comments:
  • Copy and Paste not working in the popup.
  • Documentation needs to be updated.

On Mon, Jan 11, 2021 at 4:38 PM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Hackers,

Attached patch improves the way explain plan details tooltip for a node is shown. With the change, popup with details will be shown upon clicking a node, and it will remain open until explicitly closed.

Please review.

--
Thanks,
Aditya Toshniwal
pgAdmin hacker | Sr. Software Engineer | edbpostgres.com
"Don't Complain about Heat, Plant a TREE"


--
Thanks & Regards
Akshay Joshi
pgAdmin Hacker | Principal Software Architect
EDB Postgres
Mobile: +91 976-788-8246



--
Thanks,
Aditya Toshniwal
pgAdmin hacker | Sr. Software Engineer | edbpostgres.com
"Don't Complain about Heat, Plant a TREE"
Вложения

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

Предыдущее
От: Magnus Hagander
Дата:
Сообщение: Re: [pgAdmin4][Patch] - RM 5457 - Kerberos Authentication - Phase 1
Следующее
От: Akshay Joshi
Дата:
Сообщение: pgAdmin 4 commit: Improve the explain plan details by showing popup ins