Re: [pgAdmin][RM-7149]: [React] Port preferences dialog to React.

Hi Akshay,

Please find the updated patch, resolved the following issues in the
preferences dialog.

1. Add server mode validation in the binary path.
2. Updated preferences tree rendering to avoid using the ReactDOM render.
3. Updated CSS for keyboard shortcuts checkbox border makes it
consistent with input box border.
4. Fixed jasmine test case and improved code coverage.
5. Fixed sonarqube issues.
6. Added validation to disable "*Maximum column with" option *if "*Column
**sized by" option *is set to *"Column name"* in *Query Tool -> Result
grid*.
7. Updated documentation with the latest screenshots.
8. Updated documentation to fix issue #7261
<https://redmine.postgresql.org/issues/7261>: Typo on the document.

On Mon, Mar 21, 2022 at 1:31 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:

> Thanks, the patch applied. Screenshots are still not the latest. Take all
> the screenshots with the latest changes and send them back.
>
> On Mon, Mar 21, 2022 at 10:00 AM Nikhil Mohite <
> nikhil(dot)mohite(at)enterprisedb(dot)com> wrote:
>
>> Hi Akshay,
>>
>> PFA patch v2 resolved all the review comments.
>>
>> On Wed, Mar 9, 2022 at 5:48 PM Akshay Joshi <
>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Nikhil
>>>
>>> Below are the review comments:
>>> *GUI:*
>>>
>>> - By default, all nodes should be expanded, and at least one node
>>> should be selected (Check existing behavior).
>>> - Reduce the space between the expand/collapse (>) icon and the
>>> tress node label. It is most likely icons that are taking up that space.
>>> - Unable to select parent node. With old behavior when we select the
>>> parent node its immediate first child is selected and the appropriate page
>>> is displayed on the right side.
>>> - The maximize button should be consistent with all other dialogs.
>>> - Maximize not working properly, move the dialog bit up, and then
>>> click Maximize button, dialog is not visible correctly.
>>> - The help button is missing at the bottom.
>>> - A documentation update is missing.
>>> - Keyboard Shortcuts (Browser, Debugger, Query Tool, and ERD Tool) *Fix
>>> for all*
>>> - Between the 'Key' label and control, there should be a margin.
>>> - Reduce the width of the key control.
>>> - The key control loses focus when the key is pressed. Verify old
>>> behavior.
>>> - Nodes: Can we add a message or label at the top about settings?
>>> Previously, we had a switch control with a 'Show/Hide' label that was clear
>>> to the end-user.
>>> - Dashboard -> Display -> Long-running query thresholds:
>>> - Between the 'Warning'/'Alert' label and their control, there
>>> should be a margin.
>>> - There is no '*in minutes*' label at the end.
>>> - Miscellaneous -> Themes: Themes preview is not visible.
>>> - Binary Paths:
>>> - 'Set as default' should be disabled until a path is provided.
>>> - 'Set as default' must be a radio button, only one binary path
>>> can be set to default not all.
>>> - Unable to select binary path as select dialog is hidden behind
>>> the preferences dialog. It seems the preferences dialog is the modal dialog.
>>> - Adjust the validation icon properly, space should not be there
>>> at the end.
>>> - [image: Screenshot 2022-03-09 at 4.34.08 PM.png]
>>> - Query Tool: CSV/TXT Output node should be after Auto completion.
>>> - Browser -> Tab settings and Storage -> Options have identical
>>> right side page. Remove the controls appropriately.
>>>
>>> I tried but was unable to reproduce the issue.
>>
>>>
>>> *Code:*
>>>
>>> - Add comments/function header for all the new functions created.
>>>
>>>
>>> On Mon, Mar 7, 2022 at 5:54 PM Nikhil Mohite <
>>> nikhil(dot)mohite(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Hackers,
>>>>
>>>> Please find attached the patch for RM-7149
>>>> <https://redmine.postgresql.org/issues/7149>: [React] Port preferences
>>>> dialog to React.
>>>>
>>>> --
>>>> *Thanks & Regards,*
>>>> *Nikhil Mohite*
>>>> *Senior Software Engineer.*
>>>> *EDB Postgres* <https://www.enterprisedb.com/>
>>>> *Mob.No: +91-7798364578.*
>>>>
>>>
>>>
>>> --
>>> *Thanks & Regards*
>>> *Akshay Joshi*
>>> *pgAdmin Hacker | Principal Software Architect*
>>> *EDB Postgres <http://edbpostgres.com>*
>>>
>>> *Mobile: +91 976-788-8246*
>>>
>>
>> Regards,
>> Nikhil Mohite
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
> *pgAdmin Hacker | Principal Software Architect*
> *EDB Postgres <http://edbpostgres.com>*
>
> *Mobile: +91 976-788-8246*
>
Regards,
Nikhil Mohite