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.jo...@enterprisedb.com>
wrote:
> Thanks, patch applied.
>
> On Tue, Mar 10, 2020 at 1:17 PM Pradip Parkale <
> pradip.park...@enterprisedb.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.jo...@enterprisedb.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.park...@enterprisedb.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*
>
