Re: Explain details popover changes proposal

From: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Explain details popover changes proposal
Date: 2021-01-08 10:46:09
Message-ID: CAM9w-_n82T_+H2DZaLjFS8k-FQerxQqK9duHS0Af=BXbfvgHfQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi,

On Fri, Jan 8, 2021 at 2:53 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:

>
>
> On Fri, Jan 8, 2021 at 9:17 AM Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
> wrote:
>
>>
>>
>> On Fri, 8 Jan 2021 at 14:43, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>> Hi
>>>
>>> On Fri, Jan 8, 2021 at 8:36 AM Ashesh Vashi <
>>> ashesh(dot)vashi(at)enterprisedb(dot)com> wrote:
>>>
>>>> On Fri, Jan 8, 2021 at 12:37 PM Aditya Toshniwal <
>>>> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>>>>
>>>>> Hi Hackers,
>>>>>
>>>>> I'm working on https://redmine.postgresql.org/issues/5488 and as per
>>>>> the discussion, we'll remove the current tooltip way for showing the
>>>>> details. Instead, we'll add some popover/panel upon clicking a node
>>>>> which will stay until closed.
>>>>> Below is the screenshot of partial implementation.
>>>>>
>>>> I like the idea of showing details in the pop up.
>>>>
>>>
>>> I agree, though I think the presentation could use some work to make the
>>> table-like layout a little cleaner (maybe alternating row shading or subtle
>>> cell borders or something). Perhaps check with Aradhana or Chethana for
>>> ideas?
>>>
>> Yes.

>
>>>
>>>> We also need to think about the details in the SVG file downloaded.
>>>>
>>>
>>> We ignore the popups entirely at the moment don't we? It would be good
>>> to render any that are open though.
>>>
>>
>> We have put details in the title attribute, hence - it should be visible
>> as cluttered text on hover. (Assumption - I have not tried).
>>
>
> OK, well it seems reasonable to continue to do that for all nodes.
> However, maybe we should also render any popups that are open at time of
> download - that way, printing or converting to a raster format will still
> display those popups.
>
With the new changes, the SVG tooltip will be removed completely. The new
pop up is HTML based and floats above the diagram. I mean, if you zoom
in/out the plan, it does not affect the popup.
I'm thinking to append all the tables to the download SVG and add click
events on the nodes which will scroll to that table. This is just a
thought, not sure if it will work. But that would be a separate task.

>
>
>>
>> — Ashesh
>>
>>>
>>>
>>>
>>>>
>>>> -- Ashesh
>>>>
>>>>
>>>>> Please confirm if I can proceed with full implementation.
>>>>>
>>>>> [image: Screenshot 2021-01-08 at 12.31.06 PM.png]
>>>>>
>>>>> --
>>>>> Thanks,
>>>>> Aditya Toshniwal
>>>>> pgAdmin hacker | Sr. Software Engineer | *edbpostgres.com*
>>>>> <http://edbpostgres.com>
>>>>> "Don't Complain about Heat, Plant a TREE"
>>>>>
>>>>
>>>
>>> --
>>> Dave Page
>>> Blog: http://pgsnake.blogspot.com
>>> Twitter: @pgsnake
>>>
>>> EDB: http://www.enterprisedb.com
>>>
>>> --
>>
>> --
>>
>> Thanks & Regards,
>>
>> Ashesh Vashi
>> EnterpriseDB INDIA: Enterprise PostgreSQL Company
>> <http://www.enterprisedb.com>
>>
>>
>> *http://www.linkedin.com/in/asheshvashi*
>> <http://www.linkedin.com/in/asheshvashi>
>>
>
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EDB: http://www.enterprisedb.com
>
>

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

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2021-01-08 11:00:04 Re: Explain details popover changes proposal
Previous Message Dave Page 2021-01-08 09:22:49 Re: Explain details popover changes proposal