Re: [pgAdmin4][RM#4199] ENTER in Data Filter runs query

Aditya Toshniwal <[email protected]>
Newsgroups gmane.comp.db.postgresql.pgadmin.devel
Message-ID <CAM9w-_mDurPeSW60YW5RhJEK0+dwnXT6e4OZqfF-TBP0zRz7-w@mail.gmail.com>
Hi Akshay,

Patch looks good to me.

On Wed, Sep 18, 2019 at 6:02 PM Akshay Joshi <[email protected]>
wrote:

> Hi Hackers,
>
> As per discussion, I have removed the place holder string from the dialog
> and added it as a tooltip of the "OK" button.
> I have added the documentation and screenshot.
>
> Attached is the modified patch. Please review it.
>
>
> On Tue, Sep 17, 2019 at 6:52 PM Akshay Joshi <
> [email protected]> wrote:
>
>> Attached is the modified patch to lighten the text of the place holder
>> string.
>>
>> On Tue, Sep 17, 2019 at 6:29 PM Akshay Joshi <
>> [email protected]> wrote:
>>
>>> Hi Hackers,
>>>
>>> Attached is the patch to fix RM 4199 "ENTER in Data Filter runs query".
>>> To solve this problem I have remove the key mapping of the "OK" button,
>>> which is mapped to ENTER key and provided the alternative key combination
>>> that is "*SHIFT + ENTER*".
>>>
>>> I have added Placeholder message for that in code mirror.
>>> Refer Data_Filter.png.
>>> Please review it.
>>>
>>> --
>>> *Thanks & Regards*
>>> *Akshay Joshi*
>>>
>>> *Sr. Software Architect*
>>> *EnterpriseDB Software India Private Limited*
>>> *Mobile: +91 976-788-8246*
>>>
>>
>>
>> --
>> *Thanks & Regards*
>> *Akshay Joshi*
>>
>> *Sr. Software Architect*
>> *EnterpriseDB Software India Private Limited*
>> *Mobile: +91 976-788-8246*
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
>
> *Sr. Software Architect*
> *EnterpriseDB Software India Private Limited*
> *Mobile: +91 976-788-8246*
>


-- 
Thanks and Regards,
Aditya Toshniwal
Software Engineer | EnterpriseDB India | Pune
"Don't Complain about Heat, Plant a TREE"
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.