Re: [PgAdmin][RM4237]: User can not change value of Datetime picker control using keyboard (Accessibility).

Hi Pradip,

Looks like you have repeated below code in if and else both the conditions.

let $el = this.$el.find('.datetimepicker-input');
let currdate = $el.data('datetimepicker').date().clone();

We can optimize this code by taking these both lines above if condition.

Thanks,
Khushboo

On Tue, Mar 10, 2020 at 1:24 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:

> Thanks, patch applied.
>
> On Tue, Mar 10, 2020 at 1:17 PM Pradip Parkale <
> pradip(dot)parkale(at)enterprisedb(dot)com> wrote:
>
>> Hi Akshay,
>> I have already added the code to change the time. To select the time
>> picker, the user needs to press 'Alt+T' (Option+T for mac) and then
>> Up/Down arrows to change the minutes and Alt + Up/Down to change the hours.
>>
>> On Mon, Mar 9, 2020 at 11:09 AM Akshay Joshi <
>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Pradip
>>>
>>> I have tested the patch. I am able to navigate the dates using the
>>> keyboard, but not able to select the year/month and time. Can you please
>>> work on it and send the patch again.
>>>
>>> On Sat, Mar 7, 2020 at 7:00 PM Pradip Parkale <
>>> pradip(dot)parkale(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Hackers,
>>>>
>>>> Attached is a patch to change the Datetime picker value using a
>>>> keyboard.
>>>>
>>>>
>>>> --
>>>> Thanks & Regards,
>>>> Pradip Parkale
>>>> QMG, EnterpriseDB Corporation
>>>>
>>>
>>>
>>> --
>>> *Thanks & Regards*
>>> *Akshay Joshi*
>>>
>>> *Sr. Software Architect*
>>> *EnterpriseDB Software India Private Limited*
>>> *Mobile: +91 976-788-8246*
>>>
>>
>>
>> --
>> Thanks & Regards,
>> Pradip Parkale
>> QMG, EnterpriseDB Corporation
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
>
> *Sr. Software Architect*
> *EnterpriseDB Software India Private Limited*
> *Mobile: +91 976-788-8246*
>