From: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
---|---|
To: | Pradip Parkale <pradip(dot)parkale(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin][RM6231]- Add OS, Browser details in pgAdmin About us pop-up |
Date: | 2021-05-24 11:14:53 |
Message-ID: | CANxoLDdvJewB488GhL-ByzGuVERto=3Bw=w9ObyR8meK3qDQYA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Pradip
Following are the review comments:
- Following labels should be changed
- Browser Details -> Browser
- OS -> Operating System
- Config DB -> pgAdmin Database File
- Server Configuration Text Area should be *readonly* and it should have
a "col-sm-9" class instead of "col-sm-3".
- If the user resizes the About dialog then instead of showing blank
space can we automatically resize the Server configuration text area?
- The height of the About dialog in the Server mode should be less as we
are not showing "Server Configuration".
- In Runtime, we should merge the 'Browser details' in 'NW.js version'
like *0.51.2 (Chromium 89.0.4389.114)* OR at least prefix 'Chromium'
before the version in the 'Browser details'.
- Found string difference in OS details for Runtime and Desktop mode.
- OSX: "Intel Mac OS X 10_15_7" in Runtime, while "Mac OS X 10.15.7"
in Dekstop mode.
- Windows: "Win64" in Runtime, while "Windows 10" in Desktop which
also not correct, I am using "Windows Server 2016".
- Following issue found on Firefox (First Image) configuration in a
single line, Safari (Second image) too many scroll bars.
[image: Firefox.png]. [image: Safari.png]
On Fri, May 21, 2021 at 2:23 PM Pradip Parkale <
pradip(dot)parkale(at)enterprisedb(dot)com> wrote:
> Hi Hackers,
>
> Please find the attached patch for #6231. I have added OS, NW.js, browser
> details, and some server configuration in the About pgAdmin pop-up.
>
> Server configuration won't be visible to the non-admin users in server
> mode.
> --
> Thanks & Regards,
> Pradip Parkale
> Software Engineer | EnterpriseDB Corporation
>
--
*Thanks & Regards*
*Akshay Joshi*
*pgAdmin Hacker | Principal Software Architect*
*EDB Postgres <http://edbpostgres.com>*
*Mobile: +91 976-788-8246*
From | Date | Subject | |
---|---|---|---|
Next Message | Rahul Shirsat | 2021-05-24 13:13:41 | [pgAdmin][patch] RM4064 Window maximize/restore, standardize |
Previous Message | Dave Page | 2021-05-21 09:16:32 | Re: Feature #5370 User should be able to set the binary path for each database server |