Re: Explain details popover changes proposal
On Fri, Jan 8, 2021 at 11:38 AM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi,On Fri, Jan 8, 2021 at 4:30 PM Dave Page <dpage@pgadmin.org> wrote:On Fri, Jan 8, 2021 at 10:46 AM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:Hi,On Fri, Jan 8, 2021 at 2:53 PM Dave Page <dpage@pgadmin.org> wrote:On Fri, Jan 8, 2021 at 9:17 AM Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:On Fri, 8 Jan 2021 at 14:43, Dave Page <dpage@pgadmin.org> wrote:HiOn Fri, Jan 8, 2021 at 8:36 AM Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:On Fri, Jan 8, 2021 at 12:37 PM Aditya Toshniwal <aditya.toshniwal@enterprisedb.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.Currently toolips appear only on hover, and if they are large then there is no way to see them completely. In the past, there have been attempts to fix it but still it causes issues. Correcting my previous comment, it's a html tooltip and not a SVG tooltip.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?
With the new changes, multiple issues are fixed together - tooltip will not go out of screen, user can see full details, UX improved.
Sure, the main UI would definitely be improved. I just want to make sure we don't negatively impact the download version.
--— Ashesh-- AsheshPlease confirm if I can proceed with full implementation.--Thanks,Aditya ToshniwalpgAdmin hacker | Sr. Software Engineer | edbpostgres.com"Don't Complain about Heat, Plant a TREE"------Thanks,Aditya ToshniwalpgAdmin hacker | Sr. Software Engineer | edbpostgres.com"Don't Complain about Heat, Plant a TREE"----Thanks,Aditya ToshniwalpgAdmin hacker | Sr. Software Engineer | edbpostgres.com"Don't Complain about Heat, Plant a TREE"
Вложения
В списке pgadmin-hackers по дате отправления: