From: | Neel Patel <neel(dot)patel(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4][runtime][patch]: Fix for RM#2679 |
Date: | 2017-11-21 09:02:19 |
Message-ID: | CACCA4P1SLmGPcdFiDPbSLZak1=B+UbxD+yMp7ovi3R3psJJSGw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Dave,
On Mon, Nov 20, 2017 at 8:20 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
> 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.
>
Yes. You are right. We can implement the actual underlying problem but
curious to know the difference between "_blank" and "_new" target attribute.
I didn't find any reference document for target attribute value "_new". I
searched below links.
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/A#attr-target
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2017-11-21 09:06:48 | Re: [pgAdmin4][runtime][patch]: Fix for RM#2679 |
Previous Message | Murtuza Zabuawala | 2017-11-21 07:50:39 | [pgAdmin4][Patch]: Properly quote owner in Type definition |