Re: [pgAdmin4][Patch]: RM#1243 - Columns on the Query Tool should be sizeable

Поиск
Список
Период
Сортировка
От Ashesh Vashi
Тема Re: [pgAdmin4][Patch]: RM#1243 - Columns on the Query Tool should be sizeable
Дата
Msg-id CAG7mmozLQwxOABtoFkK9S+LA2iTpgL+axYm6e3b16k_NgSJTeA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin4][Patch]: RM#1243 - Columns on the Query Tool should be sizeable  (Surinder Kumar <surinder.kumar@enterprisedb.com>)
Ответы Re: [pgAdmin4][Patch]: RM#1243 - Columns on the Query Tool should be sizeable  (Surinder Kumar <surinder.kumar@enterprisedb.com>)
Список pgadmin-hackers

On Fri, May 27, 2016 at 6:27 PM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:

Hi

Please find updated patch with following changes:
  1. Replaced the outdated code for Backgrid Sizeable Columns plugin with https://github.com/FortesSolutions/backgrid-sizeable-columns library because the updated code is available here.
  2. I have figured out to calculate offset using the parameters of listener provided, so we need not to tweak the original library code.
As per our conversation, the sizeHandler is not calculating the position without resizing.
It should calculate the position even after rendering is completed.

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company



On Fri, May 27, 2016 at 5:00 PM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:


On Fri, May 27, 2016 at 4:43 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Fri, May 27, 2016 at 12:07 PM, Surinder Kumar
<surinder.kumar@enterprisedb.com> wrote:
> Hi
>
> Added support for Columns resizable in query tool. It allows user to resize
> the table columns as much as needed.
>
> We are using Backgrid Sizeable Columns plugin:
> https://github.com/WRidder/backgrid-sizeable-columns
>
> with one change in library on line no 185:
> moved the following trigger event out of if condition:
>
> if (newWidth != oldWidth) {
>
> }
>
>
> and added one additional parameter offset.
> //Trigger event
>
> view.columns.trigger("resize", columnModel, newWidth, oldWidth, offset);
>
>
> We are listening to this trigger event in sqleditor.js file where we need
> offset
> to calculate new table width on every resize event.

Please ensure any changes to third party libraries are submitted upstream.
yes, sure. 

> hence, here is the patch attached.

ENOATTACHMENT!

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

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


--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers




--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers


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

Предыдущее
От: Surinder Kumar
Дата:
Сообщение: [pgAdmin4][Patch]: Refresh CONCURRENTLY is not supported on materialized view for pg-9.3 server
Следующее
От: Ashesh Vashi
Дата:
Сообщение: Re: JSON serialization in pgAdmin 4