Re: [pgAdmin][RM-6133]: [React] Port schema diff to React

Поиск
Список
Период
Сортировка
От Akshay Joshi
Тема Re: [pgAdmin][RM-6133]: [React] Port schema diff to React
Дата
Msg-id CANxoLDf8j44rkevz5GC61nub6YQEu+mxTrzyeh9OdADH=WtH7g@mail.gmail.com
обсуждение исходный текст
Ответ на [pgAdmin][RM-6133]: [React] Port schema diff to React  (Nikhil Mohite <nikhil.mohite@enterprisedb.com>)
Ответы Re: [pgAdmin][RM-6133]: [React] Port schema diff to React  (Nikhil Mohite <nikhil.mohite@enterprisedb.com>)
Список pgadmin-hackers
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.
  • 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.mohite@enterprisedb.com> wrote:
Hi Hackers,

Please find attached the patch for RM-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.
Mob.No: +91-7798364578.


--

Akshay Joshi

Principal Software Architect

+91 9767888246

www.enterprisedb.com

     

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Aditya Toshniwal
Дата:
Сообщение: Re: [pgAdmin][RM-6133]: [React] Port schema diff to React
Следующее
От: Yogesh Mahajan
Дата:
Сообщение: Bug #7648 - Change password fail on current 6.13 version (Test and User UI)