Re: [pgAdmin][RM5038] Select2 enhancement - Load items on demand (scroll)

From: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>
To: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Cc: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
Subject: Re: [pgAdmin][RM5038] Select2 enhancement - Load items on demand (scroll)
Date: 2020-01-06 07:05:49
Message-ID: CAM9w-_k+URLiQLoDG17p7R3xWZA3Zc+BXXszU0xzuL9xVpNd+w@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Hackers,

Attached is the updated patch to fix error mentioned. Kindly review.

On Mon, Jan 6, 2020 at 11:40 AM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:

> Hi Aditya
>
> Got below error in while opening Language dialog. Please fix it.
>
> [image: Screenshot 2020-01-06 at 11.37.49 AM.png]
>
> On Fri, Jan 3, 2020 at 3:13 PM Aditya Toshniwal <
> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>
>> Hi Hackers,
>>
>> The previous patch broke the tags and tokenizer options of select2.
>> Turned out, with the change of data adapter we also need to configure
>> adapters for tags, tokenizer and others.
>> Attached is the patch to set those to the default available in select2.
>> I've also renamed onDemandLoad to showOnScroll to avoid confusion as we
>> are showing the already present data on scroll and not loading the data on
>> demand from ajax.
>>
>> Kindly review.
>>
>> On Wed, Jan 1, 2020 at 11:25 AM Akshay Joshi <
>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>
>>> Thanks, patch applied.
>>>
>>> On Tue, Dec 24, 2019 at 1:01 PM Aditya Toshniwal <
>>> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Hackers,
>>>>
>>>> Attached is the patch to enhance select2 used in pgAdmin. The select2
>>>> dropdown will show only a few items when opened initially. On scrolling it
>>>> will show others. Searching will work as before. This will reduce the
>>>> number of DOM nodes on page.
>>>> Note that, individual controls can disable the behaviour if desired by
>>>> passing onDemandLoad:false for select2 options. It is enabled by default.
>>>>
>>>> Kindly review.
>>>>
>>>> --
>>>> Thanks and Regards,
>>>> Aditya Toshniwal
>>>> pgAdmin Hacker | Sr. Software Engineer | EnterpriseDB India | Pune
>>>> "Don't Complain about Heat, Plant a TREE"
>>>>
>>>
>>>
>>> --
>>> *Thanks & Regards*
>>> *Akshay Joshi*
>>>
>>> *Sr. Software Architect*
>>> *EnterpriseDB Software India Private Limited*
>>> *Mobile: +91 976-788-8246*
>>>
>>
>>
>> --
>> Thanks and Regards,
>> Aditya Toshniwal
>> pgAdmin Hacker | Sr. Software Engineer | EnterpriseDB India | Pune
>> "Don't Complain about Heat, Plant a TREE"
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
>
> *Sr. Software Architect*
> *EnterpriseDB Software India Private Limited*
> *Mobile: +91 976-788-8246*
>

--
Thanks and Regards,
Aditya Toshniwal
pgAdmin Hacker | Sr. Software Engineer | EnterpriseDB India | Pune
"Don't Complain about Heat, Plant a TREE"

Attachment Content-Type Size
RM5038.part2_v2.patch application/octet-stream 8.2 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Akshay Joshi 2020-01-06 07:20:08 pgAdmin 4 commit: Fixed broken tags and tokenizer options of select2. R
Previous Message Vivek Parmar 2020-01-06 06:33:48 Issue with PgAdmin