Re: Patch for Feature #1344 Can't use multiplemonitors (Windows 10) - Mailing list pgadmin-hackers

From Dave Page
Subject Re: Patch for Feature #1344 Can't use multiplemonitors (Windows 10)
Date
Msg-id CA+OCxoyf4_VFADyTaW4Hb02d1URhE-e_0C3U8SiwRdi8BkkrsA@mail.gmail.com
Whole thread Raw
In response to Re: Patch for Feature #1344 Can't use multiplemonitors (Windows 10)  (Dave Page <dpage@pgadmin.org>)
Responses Re: Patch for Feature #1344 Can't use multiplemonitors (Windows 10)
List pgadmin-hackers
Hi

I've found a bug with this. When you set either of the options to True
for the very first time, then hit OK on the Preferences dialogue, the
setting is not stored. You have to go back into Preferences and set it
a second time.

From then on, it seems to be fine, so I wonder if we're failing to
store the config value the first time we insert a row for it, but
updates to it work fine.

Can you take a look please?

Thanks.

On Fri, Mar 24, 2017 at 10:45 AM, Dave Page <dpage@pgadmin.org> wrote:
>
>
> On Fri, Mar 24, 2017 at 8:52 AM, Akshay Joshi
> <akshay.joshi@enterprisedb.com> wrote:
>>
>> Hi All
>>
>> I have implemented Feature #1344 Can't use multiple monitors (Windows 10).
>> Attached is the patch file. I did following things in this patch:
>>
>> Added "Open in New Browser Tab" preferences setting for "Query Tool" and
>> "Debugger".
>> Use "window.open" and related functions to open the query tool and
>> debugger in other browser tab.
>>
>> I have tested it on Chrome, Firefox, Safari and Runtime. In Safari user
>> will have to uncheck "Block pop up window" settings.
>
>
> Thanks, applied.
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company



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

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: [pgAdmin4][Patch]: Fixed 2190 - Move languageselection to Preferences.
Next
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Don't require an auth key for the static helpfiles.