From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Neel Patel <neel(dot)patel(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4][runtime][patch]: Fix for RM#2679 |
Date: | 2017-11-20 14:50:32 |
Message-ID: | CA+OCxozoRsQqt9Nq6o9e084kB+89_jUJN9LY_jjCdwpDhqC0+w@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi
On Mon, Nov 20, 2017 at 1:10 PM, Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
wrote:
> Hi,
>
> Please find attached patch to fix RM#2679.
>
> *Issue:-*
> Getting started links does not open second time from "Dashboard" panel if
> User close runtime tab and open any URL again.
>
> *Analysis:-*
> As in runtime Qt application, when user defined "target=_new" then
> "createWindow" signal is called but when user close that new windows and
> again click on link then "createWindow" signal is not getting called so
> from user point view nothing will happen.
>
> *Solution:-*
> To make it work in both runtime and web application, changed "target"
> attribute to "_blank" so that "createWindow" signal will be called every
> time when user click on any link.
>
I think this is a partial workaround for the problem. We link to external
sites such as postgresql.org - what happens if that tries to open something
with target="_new"? It should be expected to work as well. I think we need
to fix the underlying problem, not try to work around it.
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | pgAdmin 4 Jenkins | 2017-11-20 15:08:05 | Build failed in Jenkins: pgadmin4-master-python26 #513 |
Previous Message | Dave Page | 2017-11-20 14:43:38 | pgAdmin 4 commit: Ensure we can download large files and keep the user |