Re: [pgAdmin][RM5716] AJAX to generate the charts continues to fire after we disconnect the server

Поиск
Список
Период
Сортировка
От Aditya Toshniwal
Тема Re: [pgAdmin][RM5716] AJAX to generate the charts continues to fire after we disconnect the server
Дата
Msg-id CAM9w-_=LEdYpVXNzk25Hdrbnpe3rizhGObr8ObxfdBnFWa2hBg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin][RM5716] AJAX to generate the charts continues to fire after we disconnect the server  (Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com>)
Ответы Re: [pgAdmin][RM5716] AJAX to generate the charts continues to fire after we disconnect the server
Список pgadmin-hackers
Hi Murtuza,

On Thu, Aug 6, 2020 at 3:21 PM Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hello,

On Thu, Aug 6, 2020 at 2:32 PM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Hackers,

Attached is the patch to fix the issue mentioned in the subject.
Regarding hiding the color legend when the server is disconnected, I feel it's good to have the legend visible as it says it's a graph. Otherwise, a box with error is ambiguous. Plus, it's not going to change anyway even if graph data is available or not.

I disagree.

A simple question, as a user can you relate those red/green/blue colors with what's currently shown on the screen?
The legends are added only when graphs are rendered at least once. So if error occurs, it means the graphs data is unavailable.
And if you have noticed, there are labels - Inserts/Updates/Delete beside the colors red/green/blue.

image.png

 

Please review.

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


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

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

Предыдущее
От: Murtuza Zabuawala
Дата:
Сообщение: Re: [pgAdmin][RM5716] AJAX to generate the charts continues to fire after we disconnect the server
Следующее
От: Murtuza Zabuawala
Дата:
Сообщение: Re: [pgAdmin][RM5716] AJAX to generate the charts continues to fire after we disconnect the server