Re: RM4292 - Dark mode support for Windows/macOS

That's why it's still on my todo, but feel free to fix it if you like :-p

On Mon, Apr 13, 2020 at 12:30 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:

> No, I think I missed that.
>
> On Mon, Apr 13, 2020 at 4:57 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Did you fix the Windows installer and macOS appbundle to ship the
>> additional required files?
>>
>> On Mon, Apr 13, 2020 at 9:36 AM Akshay Joshi <
>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>
>>> Thanks, patch applied.
>>>
>>> On Fri, Apr 10, 2020 at 5:41 PM Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
>>> wrote:
>>>
>>>> Hi Dave/Akshay,
>>>>
>>>> I had manually added the theme parameter in registry but after changing
>>>> those from the control panel as suggested by Dave, all control looks fine
>>>> in dark theme.
>>>> Please find the attached patch for review.
>>>>
>>>> Thanks,
>>>> Neel Patel
>>>>
>>>> On Thu, Apr 9, 2020 at 10:26 PM Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
>>>> wrote:
>>>>
>>>>> Hi Dave,
>>>>>
>>>>> On Thu, Apr 9, 2020 at 10:19 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>>>
>>>>>> Hi
>>>>>>
>>>>>> On Thu, Apr 9, 2020 at 2:28 PM Neel Patel <
>>>>>> neel(dot)patel(at)enterprisedb(dot)com> wrote:
>>>>>>
>>>>>>> Hi Dave,
>>>>>>>
>>>>>>> I reviewed and tested the code on Windows and fixed below issues.
>>>>>>> Except below, it looks good to me.
>>>>>>>
>>>>>>> - Compilation error on windows
>>>>>>>
>>>>>>> Oops, thanks.
>>>>>>
>>>>>>>
>>>>>>> - In configuration window, "maximumSize" of the control is
>>>>>>> provided along with "minimumSize" so it prevents control from expanding
>>>>>>> when the user resizes the window. Removed maxSize and set as default.
>>>>>>>
>>>>>>> OK.
>>>>>>
>>>>>>
>>>>>>> A Couple of points for discussion.
>>>>>>>
>>>>>>> - AFAIK - Users need to manually set the "AppsUseLightTheme"
>>>>>>> value in the registry, right ? Do we need to document that somewhere ?
>>>>>>>
>>>>>>> No, it's an option if you go to personalise your desktop:
>>>>>>
>>>>>
>>>>> OK
>>>>>
>>>>>
>>>>>>
>>>>>> [image: Screenshot 2020-04-09 at 17.45.02.png]
>>>>>>
>>>>>>
>>>>>>>
>>>>>>> - As we have introduced 2 new "dark.qss" & "light.qss" files, so
>>>>>>> during packaging we need to make sure that it should reside along with
>>>>>>> "pgAdmin4" application binary otherwise those css will not be applied at
>>>>>>> runtime. right ?
>>>>>>>
>>>>>>> Hmm, that's a good point. I'll look at that.
>>>>>>
>>>>>>>
>>>>>>> - 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]
>>>>>> Thanks!
>>>>>>
>>>>>>
>>>>>>>
>>>>>>> Attached is the updated patch.
>>>>>>>
>>>>>>> Thanks,
>>>>>>> Neel Patel
>>>>>>>
>>>>>>> On Thu, Apr 9, 2020 at 11:12 AM Akshay Joshi <
>>>>>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>>>>>
>>>>>>>> Neel,
>>>>>>>>
>>>>>>>> Can you please review/test it?
>>>>>>>>
>>>>>>>> On Wed, Apr 8, 2020 at 9:08 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>>>>>>
>>>>>>>>> The attached patch detects if dark mode is enabled on Windows
>>>>>>>>> (10+)/macOS (10.14+) at server start, and styles the runtime accordingly.
>>>>>>>>> It doesn't dynamically switch if the user changes their preferences.
>>>>>>>>>
>>>>>>>>> Linux builds continue to use the standard styling from Qt.
>>>>>>>>>
>>>>>>>>> NOTE: I've tested this on macOS, but my Windows build system is
>>>>>>>>> playing up at the moment. Akshay, can you have someone on your team test it
>>>>>>>>> please? I believe the code should work; it's just a case of reading a reg
>>>>>>>>> key and then doing the same thing as on macOS to set the theme accordingly.
>>>>>>>>>
>>>>>>>>> --
>>>>>>>>> Dave Page
>>>>>>>>> Blog: http://pgsnake.blogspot.com
>>>>>>>>> Twitter: @pgsnake
>>>>>>>>>
>>>>>>>>> EnterpriseDB UK: http://www.enterprisedb.com
>>>>>>>>> The Enterprise PostgreSQL Company
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> --
>>>>>>>> *Thanks & Regards*
>>>>>>>> *Akshay Joshi*
>>>>>>>>
>>>>>>>> *Sr. Software Architect*
>>>>>>>> *EnterpriseDB Software India Private Limited*
>>>>>>>> *Mobile: +91 976-788-8246*
>>>>>>>>
>>>>>>>
>>>>>>
>>>>>> --
>>>>>> Dave Page
>>>>>> Blog: http://pgsnake.blogspot.com
>>>>>> Twitter: @pgsnake
>>>>>>
>>>>>> EnterpriseDB UK: http://www.enterprisedb.com
>>>>>> The Enterprise PostgreSQL Company
>>>>>>
>>>>>
>>>
>>> --
>>> *Thanks & Regards*
>>> *Akshay Joshi*
>>>
>>> *Sr. Software Architect*
>>> *EnterpriseDB Software India Private Limited*
>>> *Mobile: +91 976-788-8246*
>>>
>>
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
>
> *Sr. Software Architect*
> *EnterpriseDB Software India Private Limited*
> *Mobile: +91 976-788-8246*
>

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

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