Re: Explain details popover changes proposal

On Fri, Jan 8, 2021 at 10:46 AM Aditya Toshniwal <
aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:

> 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.
>

We certainly shouldn't remove functionality that works now, without
replacing it with something at least as good.

I think this is relatively low priority though - can't we just fix the
basic problem, or is that caused by the SVG tooltips, whose location is out
of our control?

>
>>
>>>
>>> — 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"
>

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EDB: http://www.enterprisedb.com