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
>
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 |