public inbox for [email protected]  

help / color / mirror / Atom feed
From: Khushboo Vashi <[email protected]>
To: Sahil Harpal <[email protected]>
Cc: [email protected]
Subject: Re: Pgadmin4 System Stats Extension Design
Date: Tue, 22 Aug 2023 16:02:28 +0530
Message-ID: <CAFOhELdJ8XB_Mp+qo0Kg7ewKoo0bWBqGFzooX3_TbfgfHr87Hg@mail.gmail.com> (raw)
In-Reply-To: <CAKi=nndFui4CJ52widumsLJL7YXqWdH0ryT-Enn_AD7AXTzdaQ@mail.gmail.com>
References: <CAKi=nnc6Ze8QWBmSHe9YBYciWeDzMf=-rw9BgyNocFSokrpi6w@mail.gmail.com>
	<CAKi=nneuTPLMppZaAm-Y1X=4mY3UnbkbJAMCHF=Fu-2+kateRg@mail.gmail.com>
	<CAFOhELdhcYr9NBNrkEAWt_cwP77U2ANRK4xonqKA1p6WpMBdMA@mail.gmail.com>
	<CAKi=nnfSoagHGwa4To7ND39Sb0qcUVBsL2HZB0X4d_4qAus2Nw@mail.gmail.com>
	<CAFOhELff7GfskXcjcW87WC9i3QL6PmPzvsMwKfwLC8ib-hKG4Q@mail.gmail.com>
	<CAKi=nndUzwyZ-YcWOS5uxU7jL-WsFNq0gZ=zr=WkHazTca4ikA@mail.gmail.com>
	<CAFOhELcVK-_OAEnc48sqCiC09Gpv5pDj3PLFw6=nZ=QrnHaGBw@mail.gmail.com>
	<CAKi=nndFui4CJ52widumsLJL7YXqWdH0ryT-Enn_AD7AXTzdaQ@mail.gmail.com>

Hi Sahil,

On Tue, Aug 22, 2023 at 1:19 PM Sahil Harpal <[email protected]>
wrote:

> Hello Khushboo,
>
> Thank you for sharing this with me. As you can see, towards the end, when
> you encountered the tooltip issue, the size of the graph container was
> changing. I even found this bug in the actual pgAdmin4 application
> installed on my system. Please see the attached video.
>
> Here's what I believe is happening: When the container's size changes, it
> seems that tooltips continue to accumulate without being properly removed.
> The method we use to display tooltips involves inserting a new element
> into the DOM with the class name "uplot-tooltip". So, if we remove all the
> existing tooltip elements before inserting a new one, it could potentially
> resolve the issue. Would like to know your thoughts on this.
>
> It would be good if you fix this issue as you already investigated and
found the root cause. But if you have other things to complete as the
deadline is approaching you can create this issue at pgAdmin repo as it is
an existing one.

Thanks,
Khushboo

> Thank you,
> Sahil
>
> On Mon, 21 Aug 2023 at 11:11, Khushboo Vashi <
> [email protected]> wrote:
>
>> See the attachment.
>>
>> On Mon, Aug 21, 2023 at 10:51 AM Sahil Harpal <[email protected]>
>> wrote:
>>
>>>  pgAdmin4-cursor-rec.mp4
>>> <https://drive.google.com/file/d/19_Fw-hvZue4_1tTED9INss33eZcfdIEd/view?usp=drive_web;
>>> Hi Khushboo,
>>> On Mon, 21 Aug 2023 at 09:55, Khushboo Vashi <
>>> [email protected]> wrote:
>>>
>>>>
>>>> On Fri, Aug 18, 2023 at 10:08 AM Sahil Harpal <
>>>> [email protected]> wrote:
>>>>
>>>>> On Thu, 17 Aug 2023 at 17:57, Khushboo Vashi <
>>>>> [email protected]> wrote:
>>>>>
>>>>>>
>>>>>>    - The Tooltip for graph points remains on the page. Please refer
>>>>>>    to the below screenshot.
>>>>>>
>>>>>> Is this happening simply by moving the cursor within the graph area
>>>>> or when resizing the screen?
>>>>>
>>>> This is happening by simply moving the cursor across the graphs on each
>>>> tab. Maybe you can try to reduce the graph refresh rate to 1 second and try.
>>>>
>>>
>>> I'm unable to reproduce the issue even after modifying the refresh rate
>>> to 1 second. I have attached a screen recording to illustrate how it is
>>> functioning on my end.
>>> Do you have any thoughts on why the cursor issue might be happening?
>>>
>>


view thread (106+ messages)  latest in thread

reply
Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected]
  Subject: Re: Pgadmin4 System Stats Extension Design
  In-Reply-To: <CAFOhELdJ8XB_Mp+qo0Kg7ewKoo0bWBqGFzooX3_TbfgfHr87Hg@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox