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

Hi Murtuza,

On Thu, Aug 6, 2020 at 3:21 PM Murtuza Zabuawala <
murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:

> Hello,
>
> On Thu, Aug 6, 2020 at 2:32 PM Aditya Toshniwal <
> aditya(dot)toshniwal(at)enterprisedb(dot)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: image.png]
>
>
>
>>
>> Please review.
>>
>> --
>> Thanks,
>> Aditya Toshniwal
>> pgAdmin hacker | Sr. Software Engineer | *edbpostgres.com*
>> <http://edbpostgres.com>
>> "Don't Complain about Heat, Plant a TREE"
>>
>

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