Re: RM4292 - Dark mode support for Windows/macOS

From: Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>, Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: RM4292 - Dark mode support for Windows/macOS
Date: 2020-04-20 13:28:56
Message-ID: CACCA4P2VRQLbhzyrWFUHXTy+Jw-+dg3nMXkruFgLJ8zLxd2Gug@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi,

In windows, I am able to reproduce it with my dev environment and after
copying "Qt5Svg.dll", it renders the checkbox and for Mac build, we should
copy theme components in "Resources" folder which might solve the problem
and update the patch accordingly.
Once snapshot build is ready, I will verify with the same. Do review it.

Thanks,
Neel Patel

On Wed, Apr 15, 2020 at 7:12 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:

>
>
> On Wed, Apr 15, 2020 at 2:11 PM Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
> wrote:
>
>> Hi Dave,
>>
>> On Wed, Apr 15, 2020 at 5:04 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>> Hi
>>>
>>> On Thu, Apr 9, 2020 at 5:56 PM Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
>>> wrote:
>>>
>>>>
>>>>
>>>>>> - Checkbox is not visible in Configure window UI as per below
>>>>>> screenshot. Are you able to see on Mac ? If no, I can take a look at it.
>>>>>>
>>>>>> [image: Screenshot 2020-04-09 at 5.05.25 PM.png]
>>>>>>
>>>>>
>>>>> No, that works on Mac. If you can look at it that would be great (are
>>>>> the SVGs from the patch in your filesystem?):
>>>>>
>>>>
>>>> Yes, all SVGs from the patch are there. I will take a look.
>>>>
>>>>
>>>>>
>>>>> [image: Screenshot 2020-04-09 at 17.47.39.png]
>>>>>
>>>>
>>> So interestingly, with the snapshot build I now see this:
>>>
>>> [image: Screenshot 2020-04-15 at 12.34.02.png]
>>>
>>> Any idea what's going on?
>>>
>>
>> All svg files and qss reside along with the application ?
>> Initially I got this issue and when I copied the qss and svg files along
>> with the application, it resolved the issue but again I will double-check
>> and update on this.
>>
>
> Actually, no - I don't see them in the appbundle.
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>

Attachment Content-Type Size
RM_5402.patch application/octet-stream 1.3 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Akshay Joshi 2020-04-20 13:46:50 pgAdmin 4 commit: Fixed an issue where schema diff marks an identical t
Previous Message Neel Patel 2020-04-20 09:36:48 [Patch][pgAdmin4]: Fix pgagent issues