From: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
---|---|
To: | Nikhil Mohite <nikhil(dot)mohite(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin][RM-6133]: [React] Port schema diff to React |
Date: | 2022-09-07 13:55:26 |
Message-ID: | CANxoLDemWPL5U0HSaTup2+2v3vo730Ei-Y=dKEUc=c9Y7eWsEQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Thanks, the patch applied.
Please send another patch with updated screenshots and the issues you have
mentioned.
On Wed, Sep 7, 2022 at 6:58 PM Nikhil Mohite <nikhil(dot)mohite(at)enterprisedb(dot)com>
wrote:
> Hi Akshay,
>
> PFA updated patch, I have resolved the review comments, there is one issue
> with resize window I will send another patch for it.
>
> On Mon, Sep 5, 2022 at 6:34 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
> wrote:
>
>> Hi Nikhil
>>
>> The review comments are below:
>>
>> - Reduce some space from source selection and target selection.
>> - Inside the select control, make the server group label bold.
>>
>> Only this review comment is remaining, I will try to add it with a resize
> window patch.
>
>>
>> - There is currently a search icon instead of a filter icon. On the
>> button, change the cursor pointer to none.
>> - Add a help button near the 'Filter' button.
>> - The spinner and comparison status string should be bolded or larger.
>> - The distance between the 'Generate Script' and 'Filter' buttons
>> should be the same, it should not change on resizing of the browser. or
>> open schema diff in a new tab.
>> - The label (Identical: 20, Different: 10, etc.) should change based
>> on the selection of the Filter. It shows all by default and does not change.
>> - Overall font size in react data grid should be increased. The
>> header should be in bold.
>> - Two vertical scroll bars are visible. Remove if possible.
>> - The components should be aligned properly after deselecting all the
>> filters. Take a look at the existing behavior.
>> - Check the resize behavior against the existing one. Try to make it
>> the same.
>> - In the DDL comparison panel, no DDL statement is displayed when the
>> 'Identical' node is selected.
>> - On selection, remove the border of the cell if we are showing the
>> selected row in a different color. Also, remove the border from the header
>> node selection.
>>
>>
>> On Mon, Sep 5, 2022 at 4:09 PM Nikhil Mohite <
>> nikhil(dot)mohite(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Hackers,
>>>
>>> Please find attached the patch for RM-6133
>>> <https://redmine.postgresql.org/issues/6133> : [React] Port schema diff
>>> to React.
>>> This patch also included the changes for removing SlickGrid from pgAdmin.
>>>
>>> --
>>> *Thanks & Regards,*
>>> *Nikhil Mohite*
>>> *Senior Software Engineer.*
>>> *EDB Postgres* <https://www.enterprisedb.com/>
>>> *Mob.No: +91-7798364578.*
>>>
>>
>>
>> --
>>
>> <http://www.enterprisedb.com>
>>
>> Akshay Joshi
>>
>> Principal Software Architect
>>
>> +91 9767888246
>>
>> www.enterprisedb.com
>>
>> <https://www.linkedin.com/company/edbpostgres>
>> <https://twitter.com/edbpostgres?lang=en>
>> <https://www.facebook.com/EDBpostgres>
>> <https://www.instagram.com/EDBpostgres/>
>>
>
> Regards,
> Nikhil Mohite
>
--
Akshay Joshi
Principal Software Architect
+91 9767888246
<https://www.linkedin.com/company/edbpostgres>
<https://twitter.com/edbpostgres?lang=en>
<https://www.facebook.com/EDBpostgres>
<https://www.instagram.com/EDBpostgres/>
From | Date | Subject | |
---|---|---|---|
Next Message | Yogesh Mahajan | 2022-09-08 06:20:54 | Bug #7656 - Query Tool JSON Results, dblclk popup - textarea does not resize with dialog |
Previous Message | Akshay Joshi | 2022-09-07 13:50:49 | pgAdmin 4 commit: 1) Port schema diff to React. Fixes #6133 |