Re: [pgAdmin4][Patch]: To fix issues in Boolean editor

From: Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][Patch]: To fix issues in Boolean editor
Date: 2017-11-17 06:46:50
Message-ID: CAKKotZQHkGrPSN2RhF_kk5H=BG-boYyp827THd94cJBuLgiEXA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Dave,

I wasn't working for me in Chrome, I almost have to click second time to
set proper value in checkbox & even after that it was not saving the
correct state properly.

And regarding issue in runtime, I checked with windows runtime and found
that Qt does not support indeterminate state for the checkbox :(
So when we set the state to false it actually is indeterminate state and if
we click one more time it sets it to false.

[image: Inline image 1]

[image: Inline image 2]

--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

On Thu, Nov 16, 2017 at 10:03 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:

>
>
> On Thu, Nov 16, 2017 at 4:30 PM, Murtuza Zabuawala <
> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>
>> Hi Dave,
>>
>> On Thu, Nov 16, 2017 at 8:59 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>> Hi
>>>
>>> On Thu, Nov 16, 2017 at 2:04 PM, Murtuza Zabuawala <
>>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi,
>>>>
>>>> PFA patch to fix the issue in boolean editor & also corrected feature
>>>> test.
>>>> RM#2848
>>>>
>>>
>>> Testing in webkit on Mac (Qt 5.8), the problem does not appear to be
>>> solved for me. I have run make bundle, and restarted the runtime multiple
>>> times.
>>>
>> ​I tested in Chrome and it was working as expected, Have you tested patch
>> with browser?​
>>
>>
>
> It works fine in Chrome... but then, it was never broken there for me!
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Neel Patel 2017-11-17 08:45:56 Re: [pgAdmin4][patch][runtime]: RM#2829, RM#2491 - pgAdmin4 crashes while saving CSV data from Query tool
Previous Message Murtuza Zabuawala 2017-11-17 03:48:19 Re: Next release